The following document contains the results of Checkstyle 11.0.1 with hbase/checkstyle.xml ruleset.
| Files | |||
|---|---|---|---|
| 5188 | 0 | 0 | 1857 |
| Category | Rule | Violations | Severity |
|---|---|---|---|
| annotation | MissingDeprecated | 12 | |
| blocks | EmptyBlock | 29 | |
| NeedBraces | 869 | ||
| coding | EmptyStatement | 18 | |
| EqualsHashCode | 3 | ||
| InnerAssignment | 35 | ||
| NoFinalizer | 1 | ||
| design | FinalClass | 231 | |
| HideUtilityClassConstructor | 85 | ||
| InterfaceIsType | 23 | ||
VisibilityModifier
|
106 | ||
| imports | AvoidStarImport | 22 | |
UnusedImports
|
5 | ||
| javadoc | JavadocTagContinuationIndentation
|
23 | |
| NonEmptyAtclauseDescription | 47 | ||
| misc | ArrayTypeStyle | 115 | |
| UpperEll | 23 | ||
| sizes | LineLength
|
149 | |
| MethodLength | 60 | ||
| whitespace | FileTabCharacter | 1 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 105). | 65 | |
| sizes | LineLength | Line is longer than 100 characters (found 105). | 65 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 141 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 165 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 217 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 218 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 284 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 285 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 339 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 339 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 81 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 328 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 337 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 401 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 512 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 622 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 706 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 839 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 172 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 174 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ServerNameIgnoreStartCodeComparator should be declared as final. | 522 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 526 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 528 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 42 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 348 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 350 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 517 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 521 | |
| blocks | EmptyBlock | Must have at least one statement. | 1030 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1245 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2042 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3373 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3375 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3406 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3445 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3447 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 336 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 338 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 473 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 477 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 488 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 573 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 588 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 604 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 604 | |
| blocks | EmptyBlock | Must have at least one statement. | 1231 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1446 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2253 | |
| blocks | EmptyBlock | Must have at least one statement. | 3503 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3810 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3812 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3845 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3885 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3887 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HTestConst should be declared as final. | 36 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CreateNamespaceAction should be declared as final. | 267 | |
| design | FinalClass | Class ModifyNamespaceAction should be declared as final. | 316 | |
| design | FinalClass | Class DeleteNamespaceAction should be declared as final. | 351 | |
| design | FinalClass | Class CreateTableAction should be declared as final. | 401 | |
| design | FinalClass | Class DisableTableAction should be declared as final. | 441 | |
| design | FinalClass | Class EnableTableAction should be declared as final. | 489 | |
| design | FinalClass | Class DeleteTableAction should be declared as final. | 537 | |
| design | FinalClass | Class AddColumnFamilyAction should be declared as final. | 579 | |
| design | FinalClass | Class AlterFamilyVersionsAction should be declared as final. | 621 | |
| design | FinalClass | Class AlterFamilyEncodingAction should be declared as final. | 670 | |
| design | FinalClass | Class DeleteColumnFamilyAction should be declared as final. | 720 | |
| design | FinalClass | Class AddRowAction should be declared as final. | 756 | |
| design | FinalClass | Class Worker should be declared as final. | 820 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 902 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 88 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 99 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'else' construct must use '{}'s. | 87 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 89 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 69 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 70 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | UpperEll | Should use uppercase 'L'. | 68 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 121 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 145 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 147 | |
| misc | UpperEll | Should use uppercase 'L'. | 282 | |
| misc | UpperEll | Should use uppercase 'L'. | 297 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ThreadResourceAnalyzer should be declared as final. | 36 | |
| design | FinalClass | Class OpenFileDescriptorResourceAnalyzer should be declared as final. | 76 | |
| design | FinalClass | Class MaxFileDescriptorResourceAnalyzer should be declared as final. | 92 | |
| design | FinalClass | Class SystemLoadAverageResourceAnalyzer should be declared as final. | 103 | |
| design | FinalClass | Class ProcessCountResourceAnalyzer should be declared as final. | 113 | |
| design | FinalClass | Class AvailableMemoryMBResourceAnalyzer should be declared as final. | 123 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 31 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 43 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 419 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 710 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 263 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 264 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 31 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'hbaseCluster' must be private and have accessor methods. | 59 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 624 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockHandler should be declared as final. | 131 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 68 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 83 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NamespaceDescriptor should be declared as final. | 35 | |
| design | FinalClass | Class Builder should be declared as final. | 159 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'duration' must be private and have accessor methods. | 242 | |
| design | VisibilityModifier | Variable 'hist' must be private and have accessor methods. | 243 | |
| design | VisibilityModifier | Variable 'numbOfReplyOverThreshold' must be private and have accessor methods. | 244 | |
| design | VisibilityModifier | Variable 'numOfReplyFromReplica' must be private and have accessor methods. | 245 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 501 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 553 | |
| sizes | LineLength | Line is longer than 100 characters (found 101). | 643 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 644 | |
| misc | UpperEll | Should use uppercase 'L'. | 791 | |
| misc | UpperEll | Should use uppercase 'L'. | 791 | |
| misc | UpperEll | Should use uppercase 'L'. | 791 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1258 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1259 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1268 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1275 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1284 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1289 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1446 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1448 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1609 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2027 | |
| misc | UpperEll | Should use uppercase 'L'. | 2264 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 2428 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2698 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2699 | |
| sizes | LineLength | Line is longer than 100 characters (found 101). | 2808 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 31 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2587 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2745 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2758 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 144 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 145 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 147 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 117). | 80 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SplitLogCounters should be declared as final. | 46 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 135 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 121 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 144 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 529 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 530 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 23 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SampleStopper should be declared as final. | 74 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 176 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestCompatibilitySingletonFactoryCallable should be declared as final. | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 175 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NonParameterizedClass should be declared as final. | 46 | |
| design | FinalClass | Class ParameterizedClassWithNoParametersMethod should be declared as final. | 55 | |
| design | FinalClass | Class InValidParameterizedClass should be declared as final. | 61 | |
| design | FinalClass | Class ValidParameterizedClass2 should be declared as final. | 87 | |
| design | FinalClass | Class ValidParameterizedClass3 should be declared as final. | 95 | |
| design | FinalClass | Class ValidParameterizedClass4 should be declared as final. | 103 | |
| design | FinalClass | Class ExtendedParameterizedClass1 should be declared as final. | 111 | |
| design | FinalClass | Class ExtendedParameterizedClass2 should be declared as final. | 119 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ReflectiveCredentialProviderClient should be declared as final. | 144 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 91 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 319 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 229 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 230 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 61 | |
| design | FinalClass | Class StoppableImplementation should be declared as final. | 157 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 332 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 384 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 641 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 662 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 668 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 669 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 206 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 232 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Noop should be declared as final. | 84 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 37 | |
| blocks | EmptyBlock | Must have at least one statement. | 190 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 110 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 119). | 80 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class BackupTestUtil should be declared as final. | 30 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HFileArchiver should be declared as final. | 66 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 369 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 823 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method TestIncBackupRestore length is 155 lines (max allowed is 150). | 134 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 43 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 127 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 43 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Javadoc comment at column 20 has parse error. Missed HTML close tag 'path-to-conf'. Sometimes it means that close tag missed for one of previous tags. | 172 | |
| javadoc | JavadocTagContinuationIndentation | Javadoc comment at column 20 has parse error. Missed HTML close tag 'path-to-conf'. Sometimes it means that close tag missed for one of previous tags. | 172 | |
| javadoc | NonEmptyAtclauseDescription | Javadoc comment at column 20 has parse error. Missed HTML close tag 'path-to-conf'. Sometimes it means that close tag missed for one of previous tags. | 172 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 94 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 112). | 739 | |
| sizes | LineLength | Line is longer than 100 characters (found 112). | 750 | |
| sizes | LineLength | Line is longer than 100 characters (found 112). | 776 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'loc' must be private and have accessor methods. | 128 | |
| design | VisibilityModifier | Variable 'actions' must be private and have accessor methods. | 130 | |
| design | VisibilityModifier | Variable 'actionsByRegion' must be private and have accessor methods. | 139 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'loc' must be private and have accessor methods. | 150 | |
| design | VisibilityModifier | Variable 'stub' must be private and have accessor methods. | 154 | |
| design | VisibilityModifier | Variable 'controller' must be private and have accessor methods. | 156 | |
| design | VisibilityModifier | Variable 'resp' must be private and have accessor methods. | 158 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 34 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ClusterStatusHandler should be declared as final. | 243 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ColumnFamilyDescriptorBuilder should be declared as final. | 46 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 352 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 371 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 381 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 80 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 111). | 506 | |
| design | FinalClass | Class RestoreBackupSystemTableProcedureBiConsumer should be declared as final. | 2799 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 346 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 355 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 368 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class RegionInfoBuilder should be declared as final. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 31 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 43 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 57 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 67 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 107 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 108 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 30 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 51 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 145 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 102). | 67 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TableDescriptorBuilder should be declared as final. | 58 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 165 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testCreateTableWithRegions length is 152 lines (max allowed is 150). | 174 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 286 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 293 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | EmptyStatement | Empty statement. | 451 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 101 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 124 | |
| blocks | EmptyBlock | Must have at least one statement. | 248 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 110 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 131 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 139 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 149 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockExecutorService should be declared as final. | 56 | |
| design | FinalClass | Class MockExceptionListener should be declared as final. | 129 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FiltersRowsLessThan5 should be declared as final. | 274 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 58 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 68 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testUnmodifiableSetters length is 187 lines (max allowed is 150). | 118 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 98 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 65 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 73 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 81 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 89 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 201 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 316 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 346 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 371 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 396 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 431 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 432 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 444 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 445 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 448 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 461 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 465 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 479 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 480 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 497 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 498 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 509 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 510 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 517 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 518 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 525 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 51 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 59 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 271 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 269 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 64 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 83 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 91 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 64 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 72 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 80 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 88 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 105 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 252 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 315 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 316 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 333 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 334 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 348 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 349 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 366 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 367 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 102). | 195 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 34 | |
| sizes | LineLength | Line is longer than 100 characters (found 103). | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 46 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 46 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 54 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 63 | |
| misc | UpperEll | Should use uppercase 'L'. | 74 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 146). | 54 | |
| sizes | LineLength | Line is longer than 100 characters (found 200). | 55 | |
| sizes | LineLength | Line is longer than 100 characters (found 144). | 75 | |
| sizes | LineLength | Line is longer than 100 characters (found 206). | 76 | |
| sizes | LineLength | Line is longer than 100 characters (found 111). | 95 | |
| sizes | LineLength | Line is longer than 100 characters (found 105). | 96 | |
| sizes | LineLength | Line is longer than 100 characters (found 124). | 149 | |
| sizes | LineLength | Line is longer than 100 characters (found 169). | 190 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 125 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'countOfRPCcalls' must be private and have accessor methods. | 54 | |
| design | VisibilityModifier | Variable 'countOfRemoteRPCcalls' must be private and have accessor methods. | 59 | |
| design | VisibilityModifier | Variable 'sumOfMillisSecBetweenNexts' must be private and have accessor methods. | 64 | |
| design | VisibilityModifier | Variable 'countOfNSRE' must be private and have accessor methods. | 70 | |
| design | VisibilityModifier | Variable 'countOfBytesInResults' must be private and have accessor methods. | 75 | |
| design | VisibilityModifier | Variable 'countOfBytesInRemoteResults' must be private and have accessor methods. | 80 | |
| design | VisibilityModifier | Variable 'countOfRegions' must be private and have accessor methods. | 86 | |
| design | VisibilityModifier | Variable 'countOfRPCRetries' must be private and have accessor methods. | 91 | |
| design | VisibilityModifier | Variable 'countOfRemoteRPCRetries' must be private and have accessor methods. | 96 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 82 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 48 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 40 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Class should be declared as final. | 42 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 47 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 51 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 119). | 30 | |
| sizes | LineLength | Line is longer than 100 characters (found 101). | 35 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 120). | 42 | |
| sizes | LineLength | Line is longer than 100 characters (found 121). | 45 | |
| sizes | LineLength | Line is longer than 100 characters (found 127). | 63 | |
| sizes | LineLength | Line is longer than 100 characters (found 155). | 64 | |
| sizes | LineLength | Line is longer than 100 characters (found 162). | 66 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'ignoreZKDeleteForTesting' must be private and have accessor methods. | 82 | |
| blocks | EmptyBlock | Must have at least one statement. | 447 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 154 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 218 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'impl' must be private and have accessor methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 134 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 663 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 30 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 98 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 109). | 686 | |
| sizes | LineLength | Line is longer than 100 characters (found 109). | 716 | |
| sizes | LineLength | Line is longer than 100 characters (found 109). | 846 | |
| sizes | LineLength | Line is longer than 100 characters (found 109). | 876 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestAbortable should be declared as final. | 51 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 346 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 499 | |
| blocks | EmptyBlock | Must have at least one statement. | 525 | |
| blocks | EmptyBlock | Must have at least one statement. | 562 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 917 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 918 | |
| design | FinalClass | Class PreWALAppendWALActionsListener should be declared as final. | 964 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 134). | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 302 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 154). | 209 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 99 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 60 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 234 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 239 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 393 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 595 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 58 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 60 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 54 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 109 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 141 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 142 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 150 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 153 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 164 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 127 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 58 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 59 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 107). | 89 | |
| sizes | LineLength | Line is longer than 100 characters (found 198). | 99 | |
| sizes | LineLength | Line is longer than 100 characters (found 198). | 100 | |
| sizes | LineLength | Line is longer than 100 characters (found 198). | 101 | |
| sizes | LineLength | Line is longer than 100 characters (found 198). | 102 | |
| sizes | LineLength | Line is longer than 100 characters (found 198). | 103 | |
| sizes | LineLength | Line is longer than 100 characters (found 198). | 104 | |
| sizes | LineLength | Line is longer than 100 characters (found 198). | 105 | |
| sizes | LineLength | Line is longer than 100 characters (found 198). | 106 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 107). | 130 | |
| sizes | LineLength | Line is longer than 100 characters (found 170). | 140 | |
| sizes | LineLength | Line is longer than 100 characters (found 157). | 141 | |
| sizes | LineLength | Line is longer than 100 characters (found 157). | 142 | |
| sizes | LineLength | Line is longer than 100 characters (found 157). | 143 | |
| sizes | LineLength | Line is longer than 100 characters (found 154). | 144 | |
| sizes | LineLength | Line is longer than 100 characters (found 154). | 145 | |
| sizes | LineLength | Line is longer than 100 characters (found 154). | 146 | |
| sizes | LineLength | Line is longer than 100 characters (found 170). | 147 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 156 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 50 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 61 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 100 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 160 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 161 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 56 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 144 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 665 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 667 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 686 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 52 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 59 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 83 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 112 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 187 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 190 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 241 | |
| sizes | MethodLength | Method sortAndMerge length is 155 lines (max allowed is 150). | 262 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 125 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 179 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 783 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 784 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 786 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 787 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 789 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 790 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 791 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 801 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 806 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 808 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 810 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 812 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 815 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 817 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 866 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 322 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 323 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 121 | |
| coding | EmptyStatement | Empty statement. | 122 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 189 | |
| coding | EmptyStatement | Empty statement. | 190 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 152 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testQualifierFilter length is 154 lines (max allowed is 150). | 740 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1760 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1779 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1797 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1825 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockNextRowFilter should be declared as final. | 965 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 137 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 72 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 80 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 88 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 96 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 152 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 237 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 247 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class InvocationRecordFilter should be declared as final. | 162 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 124 | |
| coding | EmptyStatement | Empty statement. | 125 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 197 | |
| coding | EmptyStatement | Empty statement. | 198 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 245 | |
| coding | EmptyStatement | Empty statement. | 246 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 256 | |
| coding | EmptyStatement | Empty statement. | 257 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 91 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 67 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method run length is 151 lines (max allowed is 150). | 61 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 106). | 206 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 35 | |
| annotation | MissingDeprecated | Javadoc comment at column 6 has parse error. Missed HTML close tag 'b'. Sometimes it means that close tag missed for one of previous tags. | 52 | |
| javadoc | JavadocTagContinuationIndentation | Javadoc comment at column 6 has parse error. Missed HTML close tag 'b'. Sometimes it means that close tag missed for one of previous tags. | 52 | |
| javadoc | NonEmptyAtclauseDescription | Javadoc comment at column 6 has parse error. Missed HTML close tag 'b'. Sometimes it means that close tag missed for one of previous tags. | 52 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 73 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestMetrics should be declared as final. | 79 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ReadStatistics should be declared as final. | 84 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 121 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 151 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 160 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 180 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 220 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 259 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 262 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 326 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method getScanner length is 183 lines (max allowed is 150). | 98 | |
| design | VisibilityModifier | Variable 'atEnd' must be private and have accessor methods. | 104 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 134 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 139 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 175 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 176 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 177 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 180 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 181 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 182 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ParentClass should be declared as final. | 67 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 239 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 240 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 347 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 55 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 100 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 111 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 104). | 324 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testNativeSizes length is 177 lines (max allowed is 150). | 116 | |
| sizes | MethodLength | Method testSizes length is 224 lines (max allowed is 150). | 299 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 91 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 92 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class BuilderPayloadSetter should be declared as final. | 364 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 47 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 111 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 464 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 479 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 488 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 505 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 508 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Context should be declared as final. | 99 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 409 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 212 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 102). | 99 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CopyKeyEncodingState should be declared as final. | 36 | |
| design | FinalClass | Class SeekerStateBufferedEncodedSeeker should be declared as final. | 95 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DiffSeekerStateBufferedEncodedSeeker should be declared as final. | 389 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FastDiffCompressionState should be declared as final. | 56 | |
| design | FinalClass | Class FastDiffSeekerStateBufferedEncodedSeeker should be declared as final. | 375 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SeekerStateBufferedEncodedSeeker should be declared as final. | 191 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class RowIndexEncodingState should be declared as final. | 45 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SeekerState should be declared as final. | 282 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 52 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 171 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 341 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 39 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 41 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 155 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 156 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 185 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 186 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 231 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 232 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 49 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 235 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 34 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ReadyChunk should be declared as final. | 67 | |
| sizes | LineLength | Line is longer than 100 characters (found 132). | 239 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 376 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class PrefetchedHeader should be declared as final. | 1339 | |
| sizes | MethodLength | Method readBlockDataInternal length is 178 lines (max allowed is 150). | 1719 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 63 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 781 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 783 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 784 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1075 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1112 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1114 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1257 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1281 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1469 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1564 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 167 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 211 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 213 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 217 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 560 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 573 | |
| design | FinalClass | Class KeyValueStatsCollector should be declared as final. | 667 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 723 | |
| design | FinalClass | Class Builder should be declared as final. | 749 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 39 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 60 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 95 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 96 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HFileBlockTranscoder should be declared as final. | 317 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 36 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 65 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 106 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 108 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoOpDataBlockEncoder should be declared as final. | 36 | |
| design | FinalClass | Class NoneEncodingState should be declared as final. | 41 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoOpIndexBlockEncoder should be declared as final. | 44 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 88 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 34 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 37 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 40 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 56 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 83 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 84 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 88 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 100 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 101 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 105 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MyNoOpEncodedSeeker should be declared as final. | 407 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 370 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 87 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 217 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 91 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 125 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 126 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 23 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 178 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 366 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 842 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 395 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 404 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 421 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 691 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 742 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method writeDataAndReadFromHFile length is 179 lines (max allowed is 150). | 124 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 131 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 55 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 71 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 131 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 137 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 138 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 165 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 211 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 213 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 223 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 224 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 257 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 258 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 295 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 304 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 305 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 329 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 431 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 467 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 475 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 692 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method freeSpace length is 182 lines (max allowed is 150). | 1121 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1817 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1834 | |
| coding | NoFinalizer | Avoid using finalizer method. | 1869 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2194 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2197 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FileReadAccessor should be declared as final. | 338 | |
| design | FinalClass | Class FileWriteAccessor should be declared as final. | 346 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FilePathStringPool should be declared as final. | 45 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 212 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 931 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 104 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 66 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 132 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 144 | |
| design | FinalClass | Class ByteArrayBackedNode should be declared as final. | 222 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 35 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 184 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 38 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 59 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 63 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 67 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 71 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 93 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 97 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 101 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 105 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 260 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 627 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'action' must be private and have accessor methods. | 50 | |
| design | VisibilityModifier | Variable 'error' must be private and have accessor methods. | 52 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 48 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 64 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.apache.hadoop.conf.Configuration. | 22 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 422 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 423 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 522 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 523 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class RpcServerFactory should be declared as final. | 33 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 60 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 61 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 221 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 224 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 259 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 42 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 89 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 96 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 55 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 182 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.mockito.Mockito.*. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class AbortServer should be declared as final. | 64 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HBaseMarkers should be declared as final. | 24 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HBaseTestManagerFactory should be declared as final. | 105 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 37 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 38 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 111). | 51 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 52 | |
| sizes | LineLength | Line is longer than 100 characters (found 113). | 355 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 87 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 120 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 168 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 179 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 57 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 90 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 276 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 286 | |
| misc | UpperEll | Should use uppercase 'L'. | 360 | |
| misc | UpperEll | Should use uppercase 'L'. | 364 | |
| misc | UpperEll | Should use uppercase 'L'. | 368 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 182 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 202 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 215 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 245 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 256 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 282 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 104). | 246 | |
| sizes | LineLength | Line is longer than 100 characters (found 104). | 247 | |
| sizes | LineLength | Line is longer than 100 characters (found 104). | 255 | |
| sizes | LineLength | Line is longer than 100 characters (found 108). | 258 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Driver should be declared as final. | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 86 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HFileRecordReader should be declared as final. | 71 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method createRecordWriter length is 275 lines (max allowed is 150). | 202 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 97 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 98 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 161 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 162 | |
| misc | UpperEll | Should use uppercase 'L'. | 183 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 183 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 428 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 394 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 395 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 636 | |
| sizes | LineLength | Line is longer than 100 characters (found 101). | 664 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 726 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 736 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 745 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 187 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 228 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 168 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 176 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MapreduceV1Shim should be declared as final. | 73 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 104 | |
| design | FinalClass | Class MapreduceV2Shim should be declared as final. | 126 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 158 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 153 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'else' construct must use '{}'s. | 132 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 136). | 42 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SubMapRecordReader should be declared as final. | 138 | |
| design | FinalClass | Class SubMapRecordWriter should be declared as final. | 180 | |
| design | FinalClass | Class SubMapStatusReporter should be declared as final. | 194 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MutationDeserializer should be declared as final. | 52 | |
| design | FinalClass | Class MutationSerializer should be declared as final. | 73 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NullInputSplit should be declared as final. | 65 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 129 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Result94Deserializer should be declared as final. | 77 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 115 | |
| design | FinalClass | Class ResultDeserializer should be declared as final. | 120 | |
| design | FinalClass | Class ResultSerializer should be declared as final. | 140 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Javadoc comment at column 14 has parse error. Details: mismatched input ':' expecting <EOF> while parsing JAVADOC | 46 | |
| javadoc | JavadocTagContinuationIndentation | Javadoc comment at column 14 has parse error. Details: mismatched input ':' expecting <EOF> while parsing JAVADOC | 46 | |
| javadoc | NonEmptyAtclauseDescription | Javadoc comment at column 14 has parse error. Details: mismatched input ':' expecting <EOF> while parsing JAVADOC | 46 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 81 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 607 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 74 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 174 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 175 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 598 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 626 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 737 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 769 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 879 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 880 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 929 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 997 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 111). | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 69 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.mockito.Mockito.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 172 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 244 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 356 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 717 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 737 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 739 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 936 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 940 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 367 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 368 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 149). | 106 | |
| design | FinalClass | Class SplitComparator should be declared as final. | 109 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 74 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 85 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.mockito.Mockito.*. | 23 | |
| misc | UpperEll | Should use uppercase 'L'. | 360 | |
| misc | UpperEll | Should use uppercase 'L'. | 364 | |
| misc | UpperEll | Should use uppercase 'L'. | 368 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TableInputFormatForMergeTesting should be declared as final. | 141 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 133 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ProcessTimeRangeMapper should be declared as final. | 108 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 194 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method doCommandLine length is 199 lines (max allowed is 150). | 605 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method fillUp length is 209 lines (max allowed is 150). | 85 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 111 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 867 | |
| sizes | MethodLength | Method finishActiveMasterInitialization length is 436 lines (max allowed is 150). | 967 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2035 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3668 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3670 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 3811 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3854 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 80 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 255 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 233 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 258 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 131 | |
| sizes | MethodLength | Method genAssignmentPlan length is 246 lines (max allowed is 150). | 183 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 586 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 889 | |
| sizes | MethodLength | Method main length is 160 lines (max allowed is 150). | 898 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 898 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 642 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 212 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 143). | 157 | |
| design | VisibilityModifier | Variable 'installed' must be private and have accessor methods. | 479 | |
| design | VisibilityModifier | Variable 'done' must be private and have accessor methods. | 480 | |
| design | VisibilityModifier | Variable 'error' must be private and have accessor methods. | 481 | |
| design | VisibilityModifier | Variable 'isDead' must be private and have accessor methods. | 482 | |
| design | VisibilityModifier | Variable 'last_update' must be private and have accessor methods. | 495 | |
| design | VisibilityModifier | Variable 'last_version' must be private and have accessor methods. | 496 | |
| design | VisibilityModifier | Variable 'cur_worker_name' must be private and have accessor methods. | 497 | |
| design | VisibilityModifier | Variable 'batch' must be private and have accessor methods. | 498 | |
| design | VisibilityModifier | Variable 'status' must be private and have accessor methods. | 499 | |
| design | VisibilityModifier | Variable 'incarnation' must be private and have accessor methods. | 500 | |
| design | VisibilityModifier | Variable 'unforcedResubmits' must be private and have accessor methods. | 501 | |
| design | VisibilityModifier | Variable 'resubmitThresholdReached' must be private and have accessor methods. | 502 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestChoreField should be declared as final. | 120 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockAccessController should be declared as final. | 77 | |
| design | FinalClass | Class MockVisibilityController should be declared as final. | 106 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 169 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 311 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class StoppableImplementation should be declared as final. | 664 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 73 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 81 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 89 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 119 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 135 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 161). | 1559 | |
| design | FinalClass | Class RegionMetaLoadingVisitor should be declared as final. | 1839 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 152 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 155 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 159 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 162 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 138 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 150 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 159 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 168 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 656 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 110 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 87 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 171 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 195 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method BalancerClusterState length is 234 lines (max allowed is 150). | 160 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 340 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 430 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 101). | 20 | |
| design | VisibilityModifier | Variable 'ratioThreshold' must be private and have accessor methods. | 62 | |
| design | FinalClass | Class CacheAwareCandidateGenerator should be declared as final. | 225 | |
| design | FinalClass | Class CacheAwareSkewnessCandidateGenerator should be declared as final. | 344 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 532 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 48 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FavoredNodeLocalityPicker should be declared as final. | 516 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 33 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 34 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method balanceTable length is 207 lines (max allowed is 150). | 255 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method balanceTable length is 182 lines (max allowed is 150). | 564 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 389 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 98 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 150 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 208 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ForTestRackManagerOne should be declared as final. | 210 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DirScanPool should be declared as final. | 35 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 242 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 244 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WarnOnlyStoppable should be declared as final. | 311 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class StoppableImplementation should be declared as final. | 178 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Binder should be declared as final. | 48 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Binder should be declared as final. | 54 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 722 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 235 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 207 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 210 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 237 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 341 | |
| design | FinalClass | Class TableExclusiveLock should be declared as final. | 427 | |
| design | FinalClass | Class TableSharedLock should be declared as final. | 441 | |
| design | FinalClass | Class NamespaceExclusiveLock should be declared as final. | 455 | |
| design | FinalClass | Class RegionExclusiveLock should be declared as final. | 469 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 42 | |
| sizes | LineLength | Line is longer than 100 characters (found 116). | 43 | |
| sizes | LineLength | Line is longer than 100 characters (found 103). | 46 | |
| sizes | LineLength | Line is longer than 100 characters (found 116). | 47 | |
| sizes | LineLength | Line is longer than 100 characters (found 127). | 50 | |
| sizes | LineLength | Line is longer than 100 characters (found 125). | 51 | |
| sizes | LineLength | Line is longer than 100 characters (found 115). | 54 | |
| sizes | LineLength | Line is longer than 100 characters (found 128). | 55 | |
| sizes | LineLength | Line is longer than 100 characters (found 101). | 57 | |
| sizes | LineLength | Line is longer than 100 characters (found 114). | 58 | |
| sizes | LineLength | Line is longer than 100 characters (found 127). | 59 | |
| sizes | LineLength | Line is longer than 100 characters (found 107). | 61 | |
| sizes | LineLength | Line is longer than 100 characters (found 114). | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 130 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class AddProcsWorker should be declared as final. | 201 | |
| design | FinalClass | Class PollAndLockWorker should be declared as final. | 214 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MasterProcedureTestingUtility should be declared as final. | 74 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 284 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 135 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 104). | 119 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ProcedureDescriber should be declared as final. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoopLatch should be declared as final. | 88 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 219 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 189). | 50 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method executeFromState length is 163 lines (max allowed is 150). | 141 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ScheduleServerCrashProcedure should be declared as final. | 220 | |
| design | FinalClass | Class ScheduleSCPsForUnknownServers should be declared as final. | 234 | |
| design | FinalClass | Class PrimaryNotMetaRegionSelector should be declared as final. | 262 | |
| design | FinalClass | Class ReplicaNonMetaRegionSelector should be declared as final. | 278 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 138 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 40 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 114). | 103 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method executeFromState length is 158 lines (max allowed is 150). | 233 | |
| sizes | LineLength | Line is longer than 100 characters (found 112). | 288 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 78 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 261 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1241 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1298 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1302 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 306 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SimpleGauge should be declared as final. | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 58 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method performCompaction length is 308 lines (max allowed is 150). | 300 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 114 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 46 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method chore length is 179 lines (max allowed is 150). | 82 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 123 | |
| sizes | LineLength | Line is longer than 100 characters (found 196). | 130 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'message' must be private and have accessor methods. | 91 | |
| design | VisibilityModifier | Variable 'timestamp' must be private and have accessor methods. | 92 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 124 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MonitorRunnable should be declared as final. | 318 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | UpperEll | Should use uppercase 'L'. | 114 | |
| misc | UpperEll | Should use uppercase 'L'. | 330 | |
| design | FinalClass | Class TimingResult should be declared as final. | 352 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NamedQueueRecorder should be declared as final. | 40 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | UpperEll | Should use uppercase 'L'. | 473 | |
| misc | UpperEll | Should use uppercase 'L'. | 473 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 458 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 593 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 610 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1031 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1077 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1180 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1181 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1183 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 56 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 155 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 186 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 92 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 145 | |
| coding | EmptyStatement | Empty statement. | 146 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 177 | |
| coding | EmptyStatement | Empty statement. | 178 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 326 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 347 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 193 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'baseZNode' must be private and have accessor methods. | 49 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 224 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 242 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 147 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 295 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 305 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 220 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 64 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 210 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 192 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 116 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 169 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 281 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.apache.hadoop.hbase.procedure2.TestProcedureRecovery.TestStateMachineProcedure.State. | 32 | |
| design | FinalClass | Class TestProcEnv should be declared as final. | 546 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcedureEnv should be declared as final. | 165 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 270 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.apache.hadoop.hbase.procedure2.TestYieldProcedures.TestStateMachineProcedure.State. | 32 | |
| design | FinalClass | Class TestProcEnv should be declared as final. | 183 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ProtobufMagic should be declared as final. | 25 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 41 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 70 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ProtobufMessageConverter should be declared as final. | 51 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 45 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method merge length is 175 lines (max allowed is 150). | 187 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NamedLock should be declared as final. | 625 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoopOperationQuota should be declared as final. | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoopQuotaLimiter should be declared as final. | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 33 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 128 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 487 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 462 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SpaceQuotaSnapshotNotifierFactory should be declared as final. | 28 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SpaceViolationPolicyEnforcementFactory should be declared as final. | 35 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 106). | 56 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TimeBasedLimiter should be declared as final. | 31 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 190 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 194 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 233 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 51 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 413 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 86 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 88 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 591 | |
| design | FinalClass | Class InMemoryCompactionRunnable should be declared as final. | 610 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CompactionMapper should be declared as final. | 196 | |
| design | FinalClass | Class CompactionInputFormat should be declared as final. | 229 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 66 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 125 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 169 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 284 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DataTieringManager should be declared as final. | 42 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DefaultCompactionContext should be declared as final. | 113 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 51 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 35 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method doClose length is 274 lines (max allowed is 150). | 1705 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2030 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2037 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2511 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 2909 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2978 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3081 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 4945 | |
| sizes | MethodLength | Method replayRecoveredEdits length is 201 lines (max allowed is 150). | 5681 | |
| sizes | MethodLength | Method bulkLoadHFiles length is 214 lines (max allowed is 150). | 7361 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 8585 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 8586 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 8588 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 8589 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 8601 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 8602 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 209 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 320 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 341 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 353 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 408 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 903 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1017 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1040 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1065 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1104 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'TEST_SKIP_REPORTING_TRANSITION' must be private and have accessor methods. | 255 | |
| sizes | MethodLength | Method run length is 228 lines (max allowed is 150). | 812 | |
| blocks | EmptyBlock | Must have at least one statement. | 954 | |
| sizes | MethodLength | Method startServices length is 191 lines (max allowed is 150). | 1858 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 82 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FlushHandler should be declared as final. | 309 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 113). | 307 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MetricsTableValues should be declared as final. | 397 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MetricsUserAggregateFactory should be declared as final. | 23 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 45 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 133 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 580 | |
| sizes | MethodLength | Method doNonAtomicRegionMutation length is 161 lines (max allowed is 150). | 745 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 1258 | |
| sizes | MethodLength | Method multi length is 236 lines (max allowed is 150). | 2688 | |
| sizes | MethodLength | Method scan length is 228 lines (max allowed is 150). | 3338 | |
| sizes | MethodLength | Method scan length is 263 lines (max allowed is 150). | 3575 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 111). | 94 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 95 | |
| sizes | LineLength | Line is longer than 100 characters (found 105). | 96 | |
| sizes | LineLength | Line is longer than 100 characters (found 105). | 97 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 98 | |
| sizes | LineLength | Line is longer than 100 characters (found 104). | 99 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 100 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 101 | |
| sizes | LineLength | Line is longer than 100 characters (found 104). | 102 | |
| sizes | LineLength | Line is longer than 100 characters (found 104). | 103 | |
| sizes | LineLength | Line is longer than 100 characters (found 105). | 104 | |
| sizes | LineLength | Line is longer than 100 characters (found 105). | 105 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 106 | |
| sizes | LineLength | Line is longer than 100 characters (found 103). | 107 | |
| sizes | LineLength | Line is longer than 100 characters (found 106). | 108 | |
| sizes | LineLength | Line is longer than 100 characters (found 104). | 109 | |
| sizes | LineLength | Line is longer than 100 characters (found 109). | 110 | |
| sizes | LineLength | Line is longer than 100 characters (found 106). | 111 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 89 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 204 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 215 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1390 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method nextInternal length is 217 lines (max allowed is 150). | 419 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 237 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 144). | 126 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 131). | 40 | |
| sizes | LineLength | Line is longer than 100 characters (found 133). | 42 | |
| sizes | LineLength | Line is longer than 100 characters (found 131). | 46 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 149 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 154 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 179 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 240 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 243 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 275 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 277 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 39 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CancelAnswer should be declared as final. | 48 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 | |
| design | FinalClass | Class GetFileSize should be declared as final. | 50 | |
| design | FinalClass | Class GetBulkTime should be declared as final. | 64 | |
| design | FinalClass | Class GetPathName should be declared as final. | 72 | |
| design | FinalClass | Class GetMaxTimestamp should be declared as final. | 80 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 494 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 511 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 628 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 227 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 256 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 312 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 459 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class StoreFileWriter should be declared as final. | 87 | |
| design | FinalClass | Class SingleStoreFileWriter should be declared as final. | 501 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method next length is 262 lines (max allowed is 150). | 558 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1225 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1260 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1261 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 96 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 199 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class StripeCompaction should be declared as final. | 72 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class State should be declared as final. | 86 | |
| design | VisibilityModifier | Variable 'stripeEndRows' must be private and have accessor methods. | 92 | |
| design | VisibilityModifier | Variable 'stripeFiles' must be private and have accessor methods. | 99 | |
| design | VisibilityModifier | Variable 'level0Files' must be private and have accessor methods. | 101 | |
| design | VisibilityModifier | Variable 'allFilesCached' must be private and have accessor methods. | 104 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 512 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 513 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 517 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 581 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 589 | |
| design | FinalClass | Class KeyBeforeConcatenatedLists should be declared as final. | 662 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 277 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 571 | |
| coding | EmptyStatement | Empty statement. | 572 | |
| design | FinalClass | Class WrappedRowLock should be declared as final. | 730 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 118 | |
| coding | EmptyStatement | Empty statement. | 119 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 179 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MyNoOpEncodedSeeker should be declared as final. | 887 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 296 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 165 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 180 | |
| coding | EmptyStatement | Empty statement. | 181 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 152 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 167 | |
| coding | EmptyStatement | Empty statement. | 168 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 261 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 276 | |
| coding | EmptyStatement | Empty statement. | 277 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class EnvironmentEdgeForMemstoreTest should be declared as final. | 899 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 680 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 216 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 217 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 78 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 82 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 89 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 100 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 106 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 116 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 147 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 315 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 90 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 91 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 92 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 93 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 94 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 95 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 96 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 85 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 89 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 90 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 91 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 92 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 93 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 2. | 94 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 319 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 563 | |
| design | FinalClass | Class EnvironmentEdgeForMemstoreTest should be declared as final. | 1041 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 231 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 295 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 303 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 4467 | |
| design | FinalClass | Class PutThread should be declared as final. | 4691 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 4863 | |
| blocks | EmptyBlock | Must have at least one statement. | 4926 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 5077 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 5088 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 5089 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 5107 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 5796 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 286 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 292 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 298 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 331 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 343 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 151 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 166 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 316 | |
| blocks | EmptyBlock | Must have at least one statement. | 778 | |
| blocks | EmptyBlock | Must have at least one statement. | 872 | |
| blocks | EmptyBlock | Must have at least one statement. | 876 | |
| blocks | EmptyBlock | Must have at least one statement. | 1032 | |
| blocks | EmptyBlock | Must have at least one statement. | 1443 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 185 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 185 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 248 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 248 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 254 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 254 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 274 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 275 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 380 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method checkHalfHFile length is 161 lines (max allowed is 150). | 485 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 46 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 479 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 498 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 114). | 358 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 124 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 161 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 181 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 189 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 65 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestChoreField should be declared as final. | 60 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 135 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 178 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 103 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 198 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 202 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 233 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 237 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 24 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 70 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 74 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 99 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 100 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 105 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 110 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 204 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 209 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 217 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 231 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 344 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 350 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 437 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 610 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 626 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 153 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 181 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 638 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 640 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 658 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 660 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 125 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 170 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 259 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 331 | |
| blocks | EmptyBlock | Must have at least one statement. | 441 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 236 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 260 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 762 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 789 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1037 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1060 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1061 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1102 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 320 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 631 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testSelectiveFlushWithEager length is 222 lines (max allowed is 150). | 138 | |
| sizes | MethodLength | Method testSelectiveFlushWithIndexCompaction length is 260 lines (max allowed is 150). | 364 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'totalCompactingKVs' must be private and have accessor methods. | 35 | |
| design | VisibilityModifier | Variable 'currentCompactedKVs' must be private and have accessor methods. | 37 | |
| design | VisibilityModifier | Variable 'totalCompactedSize' must be private and have accessor methods. | 39 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 60 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'maxKeyCount' must be private and have accessor methods. | 137 | |
| design | VisibilityModifier | Variable 'earliestPutTs' must be private and have accessor methods. | 139 | |
| design | VisibilityModifier | Variable 'latestPutTs' must be private and have accessor methods. | 141 | |
| design | VisibilityModifier | Variable 'maxSeqId' must be private and have accessor methods. | 143 | |
| design | VisibilityModifier | Variable 'maxMVCCReadpoint' must be private and have accessor methods. | 145 | |
| design | VisibilityModifier | Variable 'maxTagsLength' must be private and have accessor methods. | 147 | |
| design | VisibilityModifier | Variable 'minSeqIdToKeep' must be private and have accessor methods. | 149 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 216 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CurrentHourProvider should be declared as final. | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CustomCellTieredUtils should be declared as final. | 29 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 191 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 209 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 332 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 351 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 357 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 490 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 57 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 202 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 132 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'ts' must be private and have accessor methods. | 108 | |
| design | VisibilityModifier | Variable 'mvcc' must be private and have accessor methods. | 109 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class AlwaysIncludeAndSeekNextRowFilter should be declared as final. | 246 | |
| design | FinalClass | Class AlwaysIncludeFilter should be declared as final. | 290 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 356 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 365 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 247 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 151 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 43 | |
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 104 | |
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 138 | |
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 175 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 109). | 184 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 461 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 184 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 185 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WALEnvironment should be declared as final. | 50 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WALUtil should be declared as final. | 59 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ReplicationLoadSink should be declared as final. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ReplicationPeerConfig should be declared as final. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DummyHBaseReplicationEndpoint should be declared as final. | 188 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.hamcrest.MatcherAssert.*. | 20 | |
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.hamcrest.Matchers.*. | 21 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 101). | 81 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 127 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 107). | 205 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 200 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 127 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 146 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 155 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'conf' must be private and have accessor methods. | 115 | |
| blocks | EmptyBlock | Must have at least one statement. | 229 | |
| sizes | MethodLength | Method run length is 171 lines (max allowed is 150). | 241 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 148 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 149 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 289 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 289 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 472 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 104 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 121 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 283). | 151 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class EmptyCredentials should be declared as final. | 446 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 443 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class IncludeStartRowFilter should be declared as final. | 148 | |
| sizes | LineLength | Line is longer than 100 characters (found 186). | 151 | |
| coding | EqualsHashCode | Definition of 'equals()' without corresponding definition of 'hashCode()'. | 153 | |
| design | FinalClass | Class JaxbJsonProviderHolder should be declared as final. | 163 | |
| design | VisibilityModifier | Variable 'type' must be private and have accessor methods. | 173 | |
| design | VisibilityModifier | Variable 'value' must be private and have accessor methods. | 175 | |
| design | VisibilityModifier | Variable 'op' must be private and have accessor methods. | 177 | |
| design | VisibilityModifier | Variable 'type' must be private and have accessor methods. | 378 | |
| design | VisibilityModifier | Variable 'op' must be private and have accessor methods. | 380 | |
| design | VisibilityModifier | Variable 'value' must be private and have accessor methods. | 384 | |
| design | VisibilityModifier | Variable 'filters' must be private and have accessor methods. | 386 | |
| design | VisibilityModifier | Variable 'limit' must be private and have accessor methods. | 388 | |
| design | VisibilityModifier | Variable 'offset' must be private and have accessor methods. | 390 | |
| design | VisibilityModifier | Variable 'family' must be private and have accessor methods. | 392 | |
| design | VisibilityModifier | Variable 'qualifier' must be private and have accessor methods. | 394 | |
| design | VisibilityModifier | Variable 'ifMissing' must be private and have accessor methods. | 396 | |
| design | VisibilityModifier | Variable 'latestVersion' must be private and have accessor methods. | 398 | |
| design | VisibilityModifier | Variable 'minColumn' must be private and have accessor methods. | 400 | |
| design | VisibilityModifier | Variable 'minColumnInclusive' must be private and have accessor methods. | 402 | |
| design | VisibilityModifier | Variable 'maxColumn' must be private and have accessor methods. | 404 | |
| design | VisibilityModifier | Variable 'maxColumnInclusive' must be private and have accessor methods. | 406 | |
| design | VisibilityModifier | Variable 'dropDependentColumn' must be private and have accessor methods. | 408 | |
| design | VisibilityModifier | Variable 'chance' must be private and have accessor methods. | 410 | |
| design | VisibilityModifier | Variable 'prefixes' must be private and have accessor methods. | 412 | |
| design | VisibilityModifier | Variable 'timestamps' must be private and have accessor methods. | 416 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1033 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 124 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 117). | 58 | |
| sizes | LineLength | Line is longer than 100 characters (found 112). | 59 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 79 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 406 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'authenticationMethod' must be private and have accessor methods. | 36 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 217 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 299 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 40 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 166 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class AccessControlUtil should be declared as final. | 45 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method checkCoveringPermission length is 169 lines (max allowed is 150). | 436 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 550 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class PermissionCache should be declared as final. | 63 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 70 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 117 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 46 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 148 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 149 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 930 | |
| sizes | MethodLength | Method testPostGrantRevoke length is 243 lines (max allowed is 150). | 1344 | |
| sizes | MethodLength | Method testCheckPermissions length is 160 lines (max allowed is 150). | 1810 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 304 | |
| blocks | EmptyBlock | Must have at least one statement. | 330 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testCellPermissions length is 220 lines (max allowed is 150). | 151 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.apache.hadoop.hbase.client.*. | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 253 | |
| sizes | LineLength | Line is longer than 100 characters (found 106). | 254 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 255 | |
| sizes | LineLength | Line is longer than 100 characters (found 116). | 256 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testCheckPermissions length is 162 lines (max allowed is 150). | 235 | |
| sizes | MethodLength | Method testPassiveMasterOperations length is 296 lines (max allowed is 150). | 458 | |
| sizes | LineLength | Line is longer than 100 characters (found 103). | 684 | |
| sizes | LineLength | Line is longer than 100 characters (found 106). | 685 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 686 | |
| sizes | LineLength | Line is longer than 100 characters (found 116). | 687 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 232 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class InMemoryServerProviderCallbackHandler should be declared as final. | 295 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SaslGssCallbackHandler should be declared as final. | 73 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 36 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockAbortable should be declared as final. | 62 | |
| sizes | MethodLength | Method testKeyUpdate length is 151 lines (max allowed is 150). | 137 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockAbortable should be declared as final. | 54 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 41 | |
| sizes | LineLength | Line is longer than 100 characters (found 115). | 116 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 63 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 64 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 365 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 401 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 629 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 674 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 433 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 76 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 45 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 48 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testPositiveCases length is 359 lines (max allowed is 150). | 41 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testPositiveCases length is 253 lines (max allowed is 150). | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 122 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'else' construct must use '{}'s. | 148 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testDeleteColumnLatestWithNoCellVisibility length is 155 lines (max allowed is 150). | 2582 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testGroupAuths length is 239 lines (max allowed is 150). | 119 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 52 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 582 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class VisibilityLabelServiceManager should be declared as final. | 30 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class VisibilityLabelsCache should be declared as final. | 48 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TagInfo should be declared as final. | 55 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 72 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 371 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 436 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 533 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 545 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 609 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1469 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1471 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1554 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1560 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1716 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1869 | |
| sizes | LineLength | Line is longer than 100 characters (found 106). | 2097 | |
| sizes | LineLength | Line is longer than 100 characters (found 110). | 2098 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2352 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2361 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2388 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2397 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3225 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 267 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 277 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 307 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 317 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 426 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoopCustomFileGrouper should be declared as final. | 222 | |
| design | FinalClass | Class ExportMapper should be declared as final. | 253 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 363 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 364 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 672 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 706 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 709 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 713 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 716 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1034 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1035 | |
| sizes | MethodLength | Method doWork length is 192 lines (max allowed is 150). | 1215 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.apache.hadoop.hbase.snapshot.RestoreSnapshotHelper.RestoreMetaChanges. | 65 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 372 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 412 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 427 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 442 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 604 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 787 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 473 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.apache.hbase.thirdparty.org.apache.commons.cli.Options. | 54 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 138 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 139 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 140 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 343 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 489 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 348 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 478 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 146 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 556 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 589 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | UpperEll | Should use uppercase 'L'. | 640 | |
| misc | UpperEll | Should use uppercase 'L'. | 699 | |
| misc | UpperEll | Should use uppercase 'L'. | 703 | |
| misc | UpperEll | Should use uppercase 'L'. | 706 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 914 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 999 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1001 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1034 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1280 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1304 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1307 | |
| design | FinalClass | Class Print should be declared as final. | 1597 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1626 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1633 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 1634 | |
| design | FinalClass | Class Delete should be declared as final. | 1658 | |
| design | FinalClass | Class Clean should be declared as final. | 1790 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 337 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 454 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 470 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 487 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 489 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 498 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 638 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 66 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 227 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 347 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 401 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WARCReader should be declared as final. | 79 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 38 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 38 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 36 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 35 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 34 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 23 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 23 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 38 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 38 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 24 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 732 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 109 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MapreduceV1Shim should be declared as final. | 73 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 104 | |
| design | FinalClass | Class MapreduceV2Shim should be declared as final. | 126 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MissingClass should be declared as final. | 114 | |
| design | FinalClass | Class MissingClassObserver should be declared as final. | 117 | |
| design | FinalClass | Class ObsoleteMethodObserver should be declared as final. | 159 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Javadoc comment at column 20 has parse error. Missed HTML close tag 'SpanData'. Sometimes it means that close tag missed for one of previous tags. | 65 | |
| javadoc | JavadocTagContinuationIndentation | Javadoc comment at column 20 has parse error. Missed HTML close tag 'SpanData'. Sometimes it means that close tag missed for one of previous tags. | 65 | |
| javadoc | NonEmptyAtclauseDescription | Javadoc comment at column 20 has parse error. Missed HTML close tag 'SpanData'. Sometimes it means that close tag missed for one of previous tags. | 65 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SpecializedPojo1Type1 should be declared as final. | 264 | |
| design | FinalClass | Class SpecializedPojo2Type1 should be declared as final. | 335 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 281 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestTool should be declared as final. | 48 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 32 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 121 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 125 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 157 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 203 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 230 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 234 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 238 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 258 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 428 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 522 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 523 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 87 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 189 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 312 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 313 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 542 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 556 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 577 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 578 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 602 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1563 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1564 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1565 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1566 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1571 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1603 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1616 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1617 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1618 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1810 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1888 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1889 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1893 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 1894 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1921 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1995 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1997 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 1999 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2025 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2027 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 2029 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2055 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2082 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2107 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 2124 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 2135 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2148 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2161 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 33 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 360 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 370 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 371 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 372 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 373 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 374 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 375 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 376 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 377 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 51 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 51 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 52 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 129 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CoprocessorClassLoader should be declared as final. | 71 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 42 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 86 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class EncryptionTest should be declared as final. | 36 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class EnvironmentEdgeManager should be declared as final. | 56 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 511 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 621 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 969 | |
| sizes | LineLength | Line is longer than 100 characters (found 115). | 1482 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1535 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1547 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1757 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 29 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 675 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 820 | |
| design | FinalClass | Class RegionBoundariesInformation should be declared as final. | 845 | |
| design | VisibilityModifier | Variable 'regionName' must be private and have accessor methods. | 846 | |
| design | VisibilityModifier | Variable 'metaFirstKey' must be private and have accessor methods. | 847 | |
| design | VisibilityModifier | Variable 'metaLastKey' must be private and have accessor methods. | 848 | |
| design | VisibilityModifier | Variable 'storesFirstKey' must be private and have accessor methods. | 849 | |
| design | VisibilityModifier | Variable 'storesLastKey' must be private and have accessor methods. | 850 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 910 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 912 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 987 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1113 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1118 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1167 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1171 | |
| sizes | LineLength | Line is longer than 100 characters (found 118). | 1177 | |
| sizes | LineLength | Line is longer than 100 characters (found 104). | 1187 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1205 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1360 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2040 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2082 | |
| sizes | MethodLength | Method checkRegionConsistency length is 231 lines (max allowed is 150). | 2103 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2106 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2178 | |
| design | VisibilityModifier | Variable 'errorCount' must be private and have accessor methods. | 2815 | |
| sizes | LineLength | Line is longer than 100 characters (found 101). | 3479 | |
| sizes | LineLength | Line is longer than 100 characters (found 114). | 3484 | |
| sizes | LineLength | Line is longer than 100 characters (found 103). | 3499 | |
| sizes | LineLength | Line is longer than 100 characters (found 120). | 3507 | |
| sizes | LineLength | Line is longer than 100 characters (found 103). | 3511 | |
| sizes | MethodLength | Method exec length is 224 lines (max allowed is 150). | 3563 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 54 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HBaseHomePath should be declared as final. | 24 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HFileArchiveTestingUtil should be declared as final. | 41 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 133 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 135 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 188 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 189 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 197 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 196 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 197 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 200 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 52 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 51 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 52 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 255 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 181 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 305 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 40 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 305 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 104 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 104 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class GcTimes should be declared as final. | 116 | |
| design | FinalClass | Class Monitor should be declared as final. | 140 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 64 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method loadTable length is 189 lines (max allowed is 150). | 575 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 648 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 887 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 46 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 68 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 96 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 120 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 230). | 82 | |
| sizes | MethodLength | Method verifyResultAgainstDataGenerator length is 170 lines (max allowed is 150). | 315 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 534 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WroteKeysTracker should be declared as final. | 123 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 296 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 369 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 369 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 371 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 469 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 470 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 471 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 472 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 473 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 474 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 475 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 476 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 477 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 609 | |
| coding | EmptyStatement | Empty statement. | 610 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 851 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 933 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 934 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 935 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 936 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 955 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 956 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 957 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 974 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 990 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 997 | |
| coding | EmptyStatement | Empty statement. | 998 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1015 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 1016 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1105 | |
| coding | EmptyStatement | Empty statement. | 1106 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1123 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1145 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1167 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1202 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1722 | |
| coding | EmptyStatement | Empty statement. | 1723 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 71 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 72 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 73 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 93 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 94 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 95 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 41 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class RegionMover should be declared as final. | 93 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 44 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 75 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 76 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 77 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 200 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 136 | |
| sizes | MethodLength | Method rollingSplit length is 196 lines (max allowed is 150). | 400 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 518 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 679 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 689 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 757 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 758 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 123 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 138 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ShutdownHookManagerV1 should be declared as final. | 53 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 65 | |
| design | FinalClass | Class ShutdownHookManagerV2 should be declared as final. | 71 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | EqualsHashCode | Definition of 'equals()' without corresponding definition of 'hashCode()'. | 125 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 108 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 114 | |
| coding | EqualsHashCode | Definition of 'equals()' without corresponding definition of 'hashCode()'. | 207 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 162 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 168 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 194 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 200 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 122 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestAvlKeyComparator should be declared as final. | 264 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 121 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 163 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 152 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 185 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 218 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 101 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 126 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class IsFileClosedDistributedFileSystem should be declared as final. | 151 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockRackManager should be declared as final. | 160 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 96 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 36 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 94 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 64 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 66 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 68 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 35 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 34 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FakeInputSplit should be declared as final. | 67 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 35 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 58 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 223 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class IOTestWriter should be declared as final. | 231 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 241 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 243 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 448 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 506 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 132 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 237 | |
| blocks | NeedBraces | 'while' construct must use '{}'s. | 916 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 1135 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method run length is 179 lines (max allowed is 150). | 179 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 323 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 326 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 347 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 399 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 529 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method splitWAL length is 154 lines (max allowed is 150). | 280 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockAbortable should be declared as final. | 56 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoExitSecurityManager should be declared as final. | 52 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ZKMultiAbortable should be declared as final. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WarnOnlyAbortable should be declared as final. | 233 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DummySecurityConfiguration should be declared as final. | 310 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ZKMetadata should be declared as final. | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'baseZNode' must be private and have accessor methods. | 43 | |
| design | VisibilityModifier | Variable 'rsZNode' must be private and have accessor methods. | 52 | |
| design | VisibilityModifier | Variable 'drainingZNode' must be private and have accessor methods. | 54 | |
| design | VisibilityModifier | Variable 'masterAddressZNode' must be private and have accessor methods. | 56 | |
| design | VisibilityModifier | Variable 'backupMasterAddressesZNode' must be private and have accessor methods. | 58 | |
| design | VisibilityModifier | Variable 'clusterStateZNode' must be private and have accessor methods. | 60 | |
| design | VisibilityModifier | Variable 'tableZNode' must be private and have accessor methods. | 66 | |
| design | VisibilityModifier | Variable 'clusterIdZNode' must be private and have accessor methods. | 68 | |
| design | VisibilityModifier | Variable 'splitLogZNode' must be private and have accessor methods. | 70 | |
| design | VisibilityModifier | Variable 'balancerZNode' must be private and have accessor methods. | 77 | |
| design | VisibilityModifier | Variable 'regionNormalizerZNode' must be private and have accessor methods. | 84 | |
| design | VisibilityModifier | Variable 'switchZNode' must be private and have accessor methods. | 91 | |
| design | VisibilityModifier | Variable 'masterMaintZNode' must be private and have accessor methods. | 93 | |
| design | VisibilityModifier | Variable 'replicationZNode' must be private and have accessor methods. | 96 | |
| design | VisibilityModifier | Variable 'peersZNode' must be private and have accessor methods. | 98 | |
| design | VisibilityModifier | Variable 'queuesZNode' must be private and have accessor methods. | 100 | |
| design | VisibilityModifier | Variable 'hfileRefsZNode' must be private and have accessor methods. | 102 | |
| design | VisibilityModifier | Variable 'snapshotCleanupZNode' must be private and have accessor methods. | 109 |