Checkstyle Results
The following document contains the results of Checkstyle 11.0.1 with hbase/checkstyle.xml ruleset.
Summary
| Files | |||
|---|---|---|---|
| 5228 | 0 | 0 | 1861 |
Files
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| annotation | MissingDeprecated | 12 | |
| blocks | EmptyBlock | 29 | |
| NeedBraces | 869 | ||
| coding | EmptyStatement | 18 | |
| EqualsHashCode | 3 | ||
| InnerAssignment | 35 | ||
| NoFinalizer | 1 | ||
| design | FinalClass | 233 | |
| HideUtilityClassConstructor | 86 | ||
| InterfaceIsType | 23 | ||
VisibilityModifier
|
106 | ||
| imports | AvoidStarImport | 21 | |
UnusedImports
|
5 | ||
| javadoc | JavadocTagContinuationIndentation
|
23 | |
| NonEmptyAtclauseDescription | 47 | ||
| misc | ArrayTypeStyle | 115 | |
| UpperEll | 23 | ||
| sizes | LineLength
|
150 | |
| MethodLength | 61 | ||
| whitespace | FileTabCharacter | 1 |
Details
log4j2.properties
| 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 |
org/apache/hadoop/hbase/AcidGuaranteesTestTool.java
| 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 |
org/apache/hadoop/hbase/CatalogFamilyFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 81 |
org/apache/hadoop/hbase/CellUtil.java
| 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 |
org/apache/hadoop/hbase/CompoundConfiguration.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 172 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 174 |
org/apache/hadoop/hbase/DistributedHBaseCluster.java
| 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 |
org/apache/hadoop/hbase/HBaseConfiguration.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 42 |
org/apache/hadoop/hbase/HBaseTestingUtil.java
| 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. | 3378 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3380 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3411 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3450 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3452 |
org/apache/hadoop/hbase/HBaseTestingUtility.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 339 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 341 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 476 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 480 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 491 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 576 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 591 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 607 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 607 | |
| blocks | EmptyBlock | Must have at least one statement. | 1234 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1449 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2256 | |
| blocks | EmptyBlock | Must have at least one statement. | 3508 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3815 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3817 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3850 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3890 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3892 |
org/apache/hadoop/hbase/HTestConst.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HTestConst should be declared as final. | 36 |
org/apache/hadoop/hbase/IntegrationTestDDLMasterFailover.java
| 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 |
org/apache/hadoop/hbase/IntegrationTestManyRegions.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 88 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 99 |
org/apache/hadoop/hbase/IntegrationTestMetaReplicas.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'else' construct must use '{}'s. | 87 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 89 |
org/apache/hadoop/hbase/IntegrationTestMobCompaction.java
| 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 |
org/apache/hadoop/hbase/IntegrationTestRegionReplicaPerf.java
| 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 |
org/apache/hadoop/hbase/JUnitResourceCheckers.java
| 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 |
org/apache/hadoop/hbase/KeyValueTestUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 31 |
org/apache/hadoop/hbase/KeyValueUtil.java
| 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 |
org/apache/hadoop/hbase/LocalHBaseCluster.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 263 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 264 |
org/apache/hadoop/hbase/MetaMockingUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 31 |
org/apache/hadoop/hbase/MiniHBaseCluster.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'hbaseCluster' must be private and have accessor methods. | 56 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 578 |
org/apache/hadoop/hbase/MockHttpApiRule.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockHandler should be declared as final. | 131 |
org/apache/hadoop/hbase/MultithreadedTestUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 68 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 83 |
org/apache/hadoop/hbase/NamespaceDescriptor.java
| 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 |
org/apache/hadoop/hbase/PerformanceEvaluation.java
| 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 |
org/apache/hadoop/hbase/PerformanceEvaluationCommons.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 31 |
org/apache/hadoop/hbase/PrivateCellUtil.java
| 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 |
org/apache/hadoop/hbase/RegionLocations.java
| 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 |
org/apache/hadoop/hbase/ScanPerformanceEvaluation.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 117). | 80 |
org/apache/hadoop/hbase/SplitLogCounters.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SplitLogCounters should be declared as final. | 46 |
org/apache/hadoop/hbase/SplitLogTask.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 135 |
org/apache/hadoop/hbase/StripeCompactionsPerformanceEvaluation.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 121 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 144 |
org/apache/hadoop/hbase/TableName.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 529 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 530 |
org/apache/hadoop/hbase/TagBuilderFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
org/apache/hadoop/hbase/TagType.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 23 |
org/apache/hadoop/hbase/TestChoreService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SampleStopper should be declared as final. | 74 |
org/apache/hadoop/hbase/TestClientClusterStatus.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 176 |
org/apache/hadoop/hbase/TestCompatibilitySingletonFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestCompatibilitySingletonFactoryCallable should be declared as final. | 44 |
org/apache/hadoop/hbase/TestGlobalMemStoreSize.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 175 |
org/apache/hadoop/hbase/TestHBaseClassTestRule.java
| 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 |
org/apache/hadoop/hbase/TestHBaseConfiguration.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ReflectiveCredentialProviderClient should be declared as final. | 144 |
org/apache/hadoop/hbase/TestIOFencing.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 91 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 319 |
org/apache/hadoop/hbase/TestLocalHBaseCluster.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/TestMetaTableAccessor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 229 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 230 |
org/apache/hadoop/hbase/TestNodeHealthCheckChore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 61 | |
| design | FinalClass | Class StoppableImplementation should be declared as final. | 157 |
org/apache/hadoop/hbase/TestPartialResultsFromClientSide.java
| 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 |
org/apache/hadoop/hbase/TestRegionRebalancing.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 206 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 232 |
org/apache/hadoop/hbase/TestServerInternalsTracing.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Noop should be declared as final. | 84 |
org/apache/hadoop/hbase/TimestampTestBase.java
| 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 |
org/apache/hadoop/hbase/ZNodeClearer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 110 |
org/apache/hadoop/hbase/backup/BackupHFileCleaner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 119). | 80 |
org/apache/hadoop/hbase/backup/BackupTestUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class BackupTestUtil should be declared as final. | 30 |
org/apache/hadoop/hbase/backup/HFileArchiver.java
| 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 |
org/apache/hadoop/hbase/backup/TestIncrementalBackupRestore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testIncBackupRestore length is 151 lines (max allowed is 150). | 61 |
org/apache/hadoop/hbase/backup/util/BackupBoundaries.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class BackupBoundaries should be declared as final. | 35 | |
| sizes | LineLength | Line is longer than 100 characters (found 105). | 74 | |
| design | FinalClass | Class BackupBoundariesBuilder should be declared as final. | 115 |
org/apache/hadoop/hbase/chaos/ChaosConstants.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
org/apache/hadoop/hbase/chaos/ChaosService.java
| 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 |
org/apache/hadoop/hbase/chaos/factories/MonkeyConstants.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 25 |
org/apache/hadoop/hbase/chaos/policies/PeriodicPolicy.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 43 |
org/apache/hadoop/hbase/chaos/util/ChaosMonkeyRunner.java
| 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 |
org/apache/hadoop/hbase/client/Action.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 94 |
org/apache/hadoop/hbase/client/Admin.java
| 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 |
org/apache/hadoop/hbase/client/AsyncBatchRpcRetryingCaller.java
| 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 |
org/apache/hadoop/hbase/client/AsyncClientScanner.java
| 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 |
org/apache/hadoop/hbase/client/ClientInternalHelper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 34 |
org/apache/hadoop/hbase/client/ClientUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
org/apache/hadoop/hbase/client/ClusterStatusListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ClusterStatusHandler should be declared as final. | 243 |
org/apache/hadoop/hbase/client/ColumnFamilyDescriptorBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ColumnFamilyDescriptorBuilder should be declared as final. | 46 |
org/apache/hadoop/hbase/client/HConnectionTestingUtility.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 |
org/apache/hadoop/hbase/client/Mutation.java
| 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 |
org/apache/hadoop/hbase/client/Query.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 80 |
org/apache/hadoop/hbase/client/RawAsyncHBaseAdmin.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 111). | 508 | |
| design | FinalClass | Class RestoreBackupSystemTableProcedureBiConsumer should be declared as final. | 2816 |
org/apache/hadoop/hbase/client/RegionInfo.java
| 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 |
org/apache/hadoop/hbase/client/RegionInfoBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class RegionInfoBuilder should be declared as final. | 25 |
org/apache/hadoop/hbase/client/RegionInfoDisplay.java
| 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 |
org/apache/hadoop/hbase/client/RegionReplicaUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 30 |
org/apache/hadoop/hbase/client/RequestControllerFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 |
org/apache/hadoop/hbase/client/ResultScanner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 51 |
org/apache/hadoop/hbase/client/RetriesExhaustedWithDetailsException.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 145 |
org/apache/hadoop/hbase/client/SnapshotDescription.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 102). | 67 |
org/apache/hadoop/hbase/client/TableDescriptorBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TableDescriptorBuilder should be declared as final. | 58 |
org/apache/hadoop/hbase/client/TableState.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 165 |
org/apache/hadoop/hbase/client/TestAdmin.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testCreateTableWithRegions length is 152 lines (max allowed is 150). | 174 |
org/apache/hadoop/hbase/client/TestAsyncRegionAdminApi.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 286 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 293 |
org/apache/hadoop/hbase/client/TestAsyncReplicationAdminApi.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | EmptyStatement | Empty statement. | 451 |
org/apache/hadoop/hbase/client/TestAvoidCellReferencesIntoShippedBlocks.java
| 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 |
org/apache/hadoop/hbase/client/TestBlockEvictionFromClient.java
| 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 |
org/apache/hadoop/hbase/client/TestBufferedMutatorParams.java
| 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 |
org/apache/hadoop/hbase/client/TestClientSideRegionScanner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FiltersRowsLessThan5 should be declared as final. | 274 |
org/apache/hadoop/hbase/client/TestFromClientSideNoCodec.java
| 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 |
org/apache/hadoop/hbase/client/TestImmutableScan.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testUnmodifiableSetters length is 187 lines (max allowed is 150). | 118 |
org/apache/hadoop/hbase/client/TestIncrementsFromClientSide.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 98 |
org/apache/hadoop/hbase/client/TestMultipleTimestamps.java
| 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 |
org/apache/hadoop/hbase/client/TestPutWithDelete.java
| 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 |
org/apache/hadoop/hbase/client/TestReplicaWithCluster.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 271 |
org/apache/hadoop/hbase/client/TestReplicasClient.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 269 |
org/apache/hadoop/hbase/client/TestScannerTimeout.java
| 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 |
org/apache/hadoop/hbase/client/TestTimestampsFilter.java
| 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 |
org/apache/hadoop/hbase/client/coprocessor/AggregationClient.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 102). | 195 |
org/apache/hadoop/hbase/client/coprocessor/Batch.java
| 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 |
org/apache/hadoop/hbase/client/coprocessor/DoubleColumnInterpreter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 46 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
org/apache/hadoop/hbase/client/coprocessor/LongColumnInterpreter.java
| 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 |
org/apache/hadoop/hbase/client/coprocessor/package-info.java
| 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 |
org/apache/hadoop/hbase/client/locking/EntityLock.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 125 |
org/apache/hadoop/hbase/client/metrics/ScanMetrics.java
| 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 |
org/apache/hadoop/hbase/codec/BaseDecoder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 82 |
org/apache/hadoop/hbase/codec/BaseEncoder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 48 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 53 |
org/apache/hadoop/hbase/codec/CodecPerformance.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 40 |
org/apache/hadoop/hbase/conf/TestConfigKey.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Class should be declared as final. | 42 |
org/apache/hadoop/hbase/constraint/CheckConfigurationConstraint.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 47 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 51 |
org/apache/hadoop/hbase/constraint/Constraint.java
| 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 |
org/apache/hadoop/hbase/constraint/package-info.java
| 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 |
org/apache/hadoop/hbase/coordination/ZKSplitLogManagerCoordination.java
| 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 |
org/apache/hadoop/hbase/coordination/ZkSplitLogWorkerCoordination.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 154 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 218 |
org/apache/hadoop/hbase/coprocessor/BaseEnvironment.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'impl' must be private and have accessor methods. | 37 |
org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 148 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 707 |
org/apache/hadoop/hbase/coprocessor/MetricsCoprocessor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 30 |
org/apache/hadoop/hbase/coprocessor/ObserverContextImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 99 |
org/apache/hadoop/hbase/coprocessor/RegionObserver.java
| 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 |
org/apache/hadoop/hbase/coprocessor/TestCoprocessorHost.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestAbortable should be declared as final. | 51 |
org/apache/hadoop/hbase/coprocessor/TestCoprocessorInterface.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 346 |
org/apache/hadoop/hbase/coprocessor/TestRegionObserverInterface.java
| 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 |
org/apache/hadoop/hbase/coprocessor/TestRegionObserverPreFlushAndPreCompact.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 134). | 53 |
org/apache/hadoop/hbase/coprocessor/TestRegionObserverScannerOpenHook.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 302 |
org/apache/hadoop/hbase/coprocessor/package-info.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 154). | 209 |
org/apache/hadoop/hbase/errorhandling/ForeignException.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 99 |
org/apache/hadoop/hbase/errorhandling/ForeignExceptionDispatcher.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
org/apache/hadoop/hbase/errorhandling/TimeoutExceptionInjector.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 60 |
org/apache/hadoop/hbase/favored/FavoredNodeAssignmentHelper.java
| 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 |
org/apache/hadoop/hbase/favored/StartcodeAgnosticServerName.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 58 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 60 |
org/apache/hadoop/hbase/filter/ColumnCountGetFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 54 |
org/apache/hadoop/hbase/filter/ColumnPrefixFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 109 |
org/apache/hadoop/hbase/filter/ColumnRangeFilter.java
| 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 |
org/apache/hadoop/hbase/filter/CompareFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 164 |
org/apache/hadoop/hbase/filter/DependentColumnFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 127 |
org/apache/hadoop/hbase/filter/FilterAllFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 58 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 59 |
org/apache/hadoop/hbase/filter/FilterListWithAND.java
| 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 |
org/apache/hadoop/hbase/filter/FilterListWithOR.java
| 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 |
org/apache/hadoop/hbase/filter/FilterTestingCluster.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 50 |
org/apache/hadoop/hbase/filter/FilterWrapper.java
| 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 |
org/apache/hadoop/hbase/filter/FirstKeyOnlyFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 56 |
org/apache/hadoop/hbase/filter/FuzzyRowFilter.java
| 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 |
org/apache/hadoop/hbase/filter/InclusiveStopFilter.java
| 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 |
org/apache/hadoop/hbase/filter/MultiRowRangeFilter.java
| 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 |
org/apache/hadoop/hbase/filter/MultipleColumnPrefixFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 125 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 179 |
org/apache/hadoop/hbase/filter/PageFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
org/apache/hadoop/hbase/filter/ParseConstants.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 |
org/apache/hadoop/hbase/filter/ParseFilter.java
| 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 |
org/apache/hadoop/hbase/filter/SingleColumnValueFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 322 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 323 |
org/apache/hadoop/hbase/filter/TestColumnPrefixFilter.java
| 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 |
org/apache/hadoop/hbase/filter/TestColumnRangeFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/filter/TestDependentColumnFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 152 |
org/apache/hadoop/hbase/filter/TestFilter.java
| 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 |
org/apache/hadoop/hbase/filter/TestFilterList.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockNextRowFilter should be declared as final. | 965 |
org/apache/hadoop/hbase/filter/TestFilterWrapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 137 |
org/apache/hadoop/hbase/filter/TestFuzzyRowAndColumnRangeFilter.java
| 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 |
org/apache/hadoop/hbase/filter/TestFuzzyRowFilterEndToEnd.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 237 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 247 |
org/apache/hadoop/hbase/filter/TestInvocationRecordFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class InvocationRecordFilter should be declared as final. | 162 |
org/apache/hadoop/hbase/filter/TestMultipleColumnPrefixFilter.java
| 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 |
org/apache/hadoop/hbase/filter/TestRandomRowFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/filter/TestRegexComparator.java
| 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 |
org/apache/hadoop/hbase/filter/WhileMatchFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 67 |
org/apache/hadoop/hbase/hbtop/HBTop.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method run length is 151 lines (max allowed is 150). | 61 |
org/apache/hadoop/hbase/http/HttpServerFunctionalTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 44 |
org/apache/hadoop/hbase/http/ProfileServlet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 106). | 206 |
org/apache/hadoop/hbase/http/lib/TestAuthenticationFilterInitializer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/http/log/LogLevelExceptionUtils.java
| 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 |
org/apache/hadoop/hbase/http/prometheus/PrometheusHadoopServlet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 73 |
org/apache/hadoop/hbase/http/prometheus/TestPrometheusServlet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestMetrics should be declared as final. | 79 |
org/apache/hadoop/hbase/io/ByteBuffInputStream.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 62 |
org/apache/hadoop/hbase/io/FSDataInputStreamWrapper.java
| 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 |
org/apache/hadoop/hbase/io/FileLink.java
| 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 |
org/apache/hadoop/hbase/io/HalfStoreFileReader.java
| 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 |
org/apache/hadoop/hbase/io/ImmutableBytesWritable.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 139 |
org/apache/hadoop/hbase/io/Reference.java
| 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 |
org/apache/hadoop/hbase/io/TestByteBufferOutputStream.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/io/TestFSDataInputStreamWrapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ParentClass should be declared as final. | 67 |
org/apache/hadoop/hbase/io/TestFileLink.java
| 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 |
org/apache/hadoop/hbase/io/TestHFileLink.java
| 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 |
org/apache/hadoop/hbase/io/TestHalfStoreFileReader.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 104). | 324 |
org/apache/hadoop/hbase/io/TestHeapSize.java
| 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 |
org/apache/hadoop/hbase/io/TestImmutableBytesWritable.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 91 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 92 |
org/apache/hadoop/hbase/io/asyncfs/FanOutOneBlockAsyncDFSOutputSaslHelper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class BuilderPayloadSetter should be declared as final. | 364 |
org/apache/hadoop/hbase/io/compress/BlockDecompressorHelper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 32 |
org/apache/hadoop/hbase/io/compress/CodecPool.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 47 |
org/apache/hadoop/hbase/io/compress/Compression.java
| 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 |
org/apache/hadoop/hbase/io/crypto/Encryption.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Context should be declared as final. | 99 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 403 |
org/apache/hadoop/hbase/io/crypto/ManagedKeyData.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 |
org/apache/hadoop/hbase/io/crypto/aes/CryptoAES.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 212 |
org/apache/hadoop/hbase/io/crypto/tls/HBaseTrustManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 102). | 99 |
org/apache/hadoop/hbase/io/encoding/CopyKeyDataBlockEncoder.java
| 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 |
org/apache/hadoop/hbase/io/encoding/DiffKeyDeltaEncoder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DiffSeekerStateBufferedEncodedSeeker should be declared as final. | 389 |
org/apache/hadoop/hbase/io/encoding/FastDiffDeltaEncoder.java
| 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 |
org/apache/hadoop/hbase/io/encoding/PrefixKeyDeltaEncoder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SeekerStateBufferedEncodedSeeker should be declared as final. | 191 |
org/apache/hadoop/hbase/io/encoding/RowIndexCodecV1.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class RowIndexEncodingState should be declared as final. | 45 |
org/apache/hadoop/hbase/io/encoding/RowIndexSeekerV1.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SeekerState should be declared as final. | 282 |
org/apache/hadoop/hbase/io/hfile/BlockCacheUtil.java
| 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 |
org/apache/hadoop/hbase/io/hfile/BlockCachesIterator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 39 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 41 |
org/apache/hadoop/hbase/io/hfile/BlockType.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 155 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 156 |
org/apache/hadoop/hbase/io/hfile/CacheStats.java
| 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 |
org/apache/hadoop/hbase/io/hfile/CacheTestUtils.java
| 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 |
org/apache/hadoop/hbase/io/hfile/CacheableDeserializerIdManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 32 |
org/apache/hadoop/hbase/io/hfile/ChecksumUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 34 |
org/apache/hadoop/hbase/io/hfile/CompoundBloomFilterWriter.java
| 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 |
org/apache/hadoop/hbase/io/hfile/FixedFileTrailer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 412 |
org/apache/hadoop/hbase/io/hfile/HFileBlock.java
| 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 |
org/apache/hadoop/hbase/io/hfile/HFileBlockIndex.java
| 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 |
org/apache/hadoop/hbase/io/hfile/HFilePrettyPrinter.java
| 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 |
org/apache/hadoop/hbase/io/hfile/KVGenerator.java
| 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 |
org/apache/hadoop/hbase/io/hfile/LruCachedBlock.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 96 |
org/apache/hadoop/hbase/io/hfile/MemcachedBlockCache.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HFileBlockTranscoder should be declared as final. | 317 |
org/apache/hadoop/hbase/io/hfile/NanoTimer.java
| 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 |
org/apache/hadoop/hbase/io/hfile/NoOpDataBlockEncoder.java
| 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 |
org/apache/hadoop/hbase/io/hfile/NoOpIndexBlockEncoder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoOpIndexBlockEncoder should be declared as final. | 44 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 88 |
org/apache/hadoop/hbase/io/hfile/RandomKeyValueUtil.java
| 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 |
org/apache/hadoop/hbase/io/hfile/TestBytesReadFromFs.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MyNoOpEncodedSeeker should be declared as final. | 407 |
org/apache/hadoop/hbase/io/hfile/TestCacheConfig.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 370 |
org/apache/hadoop/hbase/io/hfile/TestChecksum.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 87 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 217 |
org/apache/hadoop/hbase/io/hfile/TestFixedFileTrailer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 91 |
org/apache/hadoop/hbase/io/hfile/TestForceCacheImportantBlocks.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 125 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 126 |
org/apache/hadoop/hbase/io/hfile/TestHFileBlock.java
| 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 |
org/apache/hadoop/hbase/io/hfile/TestHFileBlockIndex.java
| 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 |
org/apache/hadoop/hbase/io/hfile/TestHFileWriterV3.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method writeDataAndReadFromHFile length is 179 lines (max allowed is 150). | 124 |
org/apache/hadoop/hbase/io/hfile/TestScannerSelectionUsingKeyRange.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 131 |
org/apache/hadoop/hbase/io/hfile/bucket/BucketAllocator.java
| 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 |
org/apache/hadoop/hbase/io/hfile/bucket/BucketCache.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method freeSpace length is 182 lines (max allowed is 150). | 1115 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1823 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 1839 | |
| coding | NoFinalizer | Avoid using finalizer method. | 1873 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2190 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2193 |
org/apache/hadoop/hbase/io/hfile/bucket/FileIOEngine.java
| 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 |
org/apache/hadoop/hbase/io/hfile/bucket/TestBucketCache.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 214 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 902 |
org/apache/hadoop/hbase/io/hfile/bucket/TestBucketWriterThread.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 104 |
org/apache/hadoop/hbase/io/util/LRUDictionary.java
| 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 |
org/apache/hadoop/hbase/io/util/MemorySizeUtil.java
| 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 |
org/apache/hadoop/hbase/io/util/StreamUtils.java
| 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 |
org/apache/hadoop/hbase/ipc/BlockingRpcConnection.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 628 |
org/apache/hadoop/hbase/ipc/BufferCallBeforeInitHandler.java
| 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 |
org/apache/hadoop/hbase/ipc/BufferChain.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 48 |
org/apache/hadoop/hbase/ipc/IPCUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 64 |
org/apache/hadoop/hbase/ipc/RpcCall.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.apache.hadoop.conf.Configuration. | 22 |
org/apache/hadoop/hbase/ipc/RpcServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 425 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 426 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 525 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 526 |
org/apache/hadoop/hbase/ipc/RpcServerFactory.java
| 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 |
org/apache/hadoop/hbase/ipc/SimpleRpcServer.java
| 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 |
org/apache/hadoop/hbase/ipc/SimpleRpcServerResponder.java
| 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 |
org/apache/hadoop/hbase/ipc/SimpleServerCall.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 37 |
org/apache/hadoop/hbase/ipc/SimpleServerRpcConnection.java
| 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 |
org/apache/hadoop/hbase/ipc/TestRpcHandlerException.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class AbortServer should be declared as final. | 64 |
org/apache/hadoop/hbase/keymeta/KeymetaTableAccessor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 |
org/apache/hadoop/hbase/log/HBaseMarkers.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HBaseMarkers should be declared as final. | 24 |
org/apache/hadoop/hbase/logging/HBaseTestAppender.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HBaseTestManagerFactory should be declared as final. | 105 |
org/apache/hadoop/hbase/mapred/Driver.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 |
org/apache/hadoop/hbase/mapred/HRegionPartitioner.java
| 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 |
org/apache/hadoop/hbase/mapred/MultiTableSnapshotInputFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 111). | 51 |
org/apache/hadoop/hbase/mapred/TableMapReduceUtil.java
| 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 |
org/apache/hadoop/hbase/mapred/TestGroupingTableMap.java
| 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 |
org/apache/hadoop/hbase/mapred/TestIdentityTableMap.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 57 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
org/apache/hadoop/hbase/mapred/TestRowCounter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 90 |
org/apache/hadoop/hbase/mapred/TestTableInputFormat.java
| 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 |
org/apache/hadoop/hbase/mapred/TestTableMapReduceUtil.java
| 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 |
org/apache/hadoop/hbase/mapreduce/CellCounter.java
| 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 |
org/apache/hadoop/hbase/mapreduce/CopyTable.java
| 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 |
org/apache/hadoop/hbase/mapreduce/Driver.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Driver should be declared as final. | 32 |
org/apache/hadoop/hbase/mapreduce/ExportUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 86 |
org/apache/hadoop/hbase/mapreduce/HFileInputFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HFileRecordReader should be declared as final. | 71 |
org/apache/hadoop/hbase/mapreduce/HFileOutputFormat2.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method createRecordWriter length is 275 lines (max allowed is 150). | 202 |
org/apache/hadoop/hbase/mapreduce/HFileOutputFormat2TestBase.java
| 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 |
org/apache/hadoop/hbase/mapreduce/Import.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 183 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 428 |
org/apache/hadoop/hbase/mapreduce/ImportTsv.java
| 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 |
org/apache/hadoop/hbase/mapreduce/IntegrationTestBulkLoad.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 187 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 228 |
org/apache/hadoop/hbase/mapreduce/IntegrationTestImportTsv.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 168 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 176 |
org/apache/hadoop/hbase/mapreduce/MapreduceTestingShim.java
| 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 |
org/apache/hadoop/hbase/mapreduce/MultiTableInputFormatBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 158 |
org/apache/hadoop/hbase/mapreduce/MultiTableInputFormatTestBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 153 |
org/apache/hadoop/hbase/mapreduce/MultiTableOutputFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'else' construct must use '{}'s. | 132 |
org/apache/hadoop/hbase/mapreduce/MultiTableSnapshotInputFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 136). | 42 |
org/apache/hadoop/hbase/mapreduce/MultithreadedTableMapper.java
| 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 |
org/apache/hadoop/hbase/mapreduce/MutationSerialization.java
| 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 |
org/apache/hadoop/hbase/mapreduce/NMapInputFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NullInputSplit should be declared as final. | 65 |
org/apache/hadoop/hbase/mapreduce/PutSortReducer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 129 |
org/apache/hadoop/hbase/mapreduce/ResultSerialization.java
| 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 |
org/apache/hadoop/hbase/mapreduce/RoundRobinTableInputFormat.java
| 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 |
org/apache/hadoop/hbase/mapreduce/SimpleTotalOrderPartitioner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 81 |
org/apache/hadoop/hbase/mapreduce/TableInputFormatBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 607 |
org/apache/hadoop/hbase/mapreduce/TableMapReduceUtil.java
| 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 |
org/apache/hadoop/hbase/mapreduce/TableSnapshotInputFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 111). | 53 |
org/apache/hadoop/hbase/mapreduce/TableSnapshotInputFormatImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 69 |
org/apache/hadoop/hbase/mapreduce/TestGroupingTableMapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.mockito.Mockito.*. | 20 |
org/apache/hadoop/hbase/mapreduce/TestHFileOutputFormat2.java
| 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 |
org/apache/hadoop/hbase/mapreduce/TestImportTSVWithVisibilityLabels.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 367 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 368 |
org/apache/hadoop/hbase/mapreduce/TestRoundRobinTableInputFormat.java
| 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 |
org/apache/hadoop/hbase/mapreduce/TestRowCounter.java
| 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 |
org/apache/hadoop/hbase/mapreduce/TestTableInputFormat.java
| 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 |
org/apache/hadoop/hbase/mapreduce/TestTableInputFormatBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TableInputFormatForMergeTesting should be declared as final. | 141 |
org/apache/hadoop/hbase/mapreduce/TestTableInputFormatScanBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 133 |
org/apache/hadoop/hbase/mapreduce/TestTimeRangeMapRed.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ProcessTimeRangeMapper should be declared as final. | 108 |
org/apache/hadoop/hbase/mapreduce/TextSortReducer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 194 |
org/apache/hadoop/hbase/mapreduce/replication/VerifyReplication.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method doCommandLine length is 199 lines (max allowed is 150). | 605 |
org/apache/hadoop/hbase/master/AssignmentVerificationReport.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method fillUp length is 209 lines (max allowed is 150). | 85 |
org/apache/hadoop/hbase/master/ClusterSchemaServiceImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 111 |
org/apache/hadoop/hbase/master/HMaster.java
| 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. | 2045 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3683 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3685 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 3826 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3869 |
org/apache/hadoop/hbase/master/HMasterCommandLine.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 80 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 255 |
org/apache/hadoop/hbase/master/MasterWalManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 233 |
org/apache/hadoop/hbase/master/MockRegionServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 265 |
org/apache/hadoop/hbase/master/RegionPlacementMaintainer.java
| 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 |
org/apache/hadoop/hbase/master/ServerManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 642 |
org/apache/hadoop/hbase/master/SnapshotOfRegionAssignmentFromMeta.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 212 |
org/apache/hadoop/hbase/master/SplitLogManager.java
| 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 |
org/apache/hadoop/hbase/master/TestHMasterCommandLine.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/master/TestMasterChoreScheduled.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestChoreField should be declared as final. | 120 |
org/apache/hadoop/hbase/master/TestMasterCoprocessorServices.java
| 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 |
org/apache/hadoop/hbase/master/TestMasterOperationsForRegionReplicas.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 169 |
org/apache/hadoop/hbase/master/TestMasterTransitions.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 311 |
org/apache/hadoop/hbase/master/TestRegionsRecoveryChore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class StoppableImplementation should be declared as final. | 664 |
org/apache/hadoop/hbase/master/TestWarmupRegion.java
| 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 |
org/apache/hadoop/hbase/master/assignment/AssignProcedure.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 135 |
org/apache/hadoop/hbase/master/assignment/AssignmentManager.java
| 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 |
org/apache/hadoop/hbase/master/assignment/RegionStateStore.java
| 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 |
org/apache/hadoop/hbase/master/assignment/RegionStates.java
| 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 |
org/apache/hadoop/hbase/master/assignment/TestAssignmentOnRSCrash.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 110 |
org/apache/hadoop/hbase/master/assignment/TestRegionStates.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 87 |
org/apache/hadoop/hbase/master/assignment/TestSplitTableRegionProcedure.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 171 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 195 |
org/apache/hadoop/hbase/master/balancer/BalancerClusterState.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method BalancerClusterState length is 234 lines (max allowed is 150). | 160 |
org/apache/hadoop/hbase/master/balancer/BalancerTestBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 340 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 430 |
org/apache/hadoop/hbase/master/balancer/CacheAwareLoadBalancer.java
| 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. | 61 | |
| design | FinalClass | Class CacheAwareCandidateGenerator should be declared as final. | 224 | |
| design | FinalClass | Class CacheAwareSkewnessCandidateGenerator should be declared as final. | 338 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 526 |
org/apache/hadoop/hbase/master/balancer/ClusterLoadState.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 48 |
org/apache/hadoop/hbase/master/balancer/FavoredStochasticBalancer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FavoredNodeLocalityPicker should be declared as final. | 516 |
org/apache/hadoop/hbase/master/balancer/RegionInfoComparator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 33 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 34 |
org/apache/hadoop/hbase/master/balancer/SimpleLoadBalancer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method balanceTable length is 207 lines (max allowed is 150). | 255 |
org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method balanceTable length is 182 lines (max allowed is 150). | 564 |
org/apache/hadoop/hbase/master/balancer/TestBaseLoadBalancer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 389 |
org/apache/hadoop/hbase/master/balancer/TestSimpleLoadBalancer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 98 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 150 |
org/apache/hadoop/hbase/master/balancer/TestStochasticBalancerJmxMetrics.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 208 |
org/apache/hadoop/hbase/master/balancer/TestStochasticLoadBalancerRegionReplica.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ForTestRackManagerOne should be declared as final. | 210 |
org/apache/hadoop/hbase/master/cleaner/DirScanPool.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DirScanPool should be declared as final. | 35 |
org/apache/hadoop/hbase/master/cleaner/TestHFileCleaner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 242 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 244 |
org/apache/hadoop/hbase/master/cleaner/TestReplicationBarrierCleaner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WarnOnlyStoppable should be declared as final. | 311 |
org/apache/hadoop/hbase/master/cleaner/TestSnapshotCleanerChore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class StoppableImplementation should be declared as final. | 178 |
org/apache/hadoop/hbase/master/http/gson/GsonSerializationFeature.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Binder should be declared as final. | 48 |
org/apache/hadoop/hbase/master/http/jersey/MasterFeature.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class Binder should be declared as final. | 54 |
org/apache/hadoop/hbase/master/janitor/TestCatalogJanitor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 722 |
org/apache/hadoop/hbase/master/locking/LockManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 235 |
org/apache/hadoop/hbase/master/locking/LockProcedure.java
| 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 |
org/apache/hadoop/hbase/master/normalizer/package-info.java
| 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 |
org/apache/hadoop/hbase/master/procedure/MasterProcedureEnv.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 130 |
org/apache/hadoop/hbase/master/procedure/MasterProcedureSchedulerPerformanceEvaluation.java
| 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 |
org/apache/hadoop/hbase/master/procedure/MasterProcedureTestingUtility.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MasterProcedureTestingUtility should be declared as final. | 74 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 284 |
org/apache/hadoop/hbase/master/procedure/MasterProcedureUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 135 |
org/apache/hadoop/hbase/master/procedure/ModifyTableProcedure.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 104). | 121 |
org/apache/hadoop/hbase/master/procedure/ProcedureDescriber.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ProcedureDescriber should be declared as final. | 37 |
org/apache/hadoop/hbase/master/procedure/ProcedurePrepareLatch.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoopLatch should be declared as final. | 88 |
org/apache/hadoop/hbase/master/procedure/ProcedureSyncWait.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 219 |
org/apache/hadoop/hbase/master/procedure/ReloadQuotasProcedure.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 189). | 50 |
org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method executeFromState length is 163 lines (max allowed is 150). | 141 |
org/apache/hadoop/hbase/master/procedure/TestHBCKSCP.java
| 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 |
org/apache/hadoop/hbase/master/procedure/TestMasterProcedureEvents.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 138 |
org/apache/hadoop/hbase/master/region/MasterRegionFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 40 |
org/apache/hadoop/hbase/master/replication/RemovePeerProcedure.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 114). | 103 |
org/apache/hadoop/hbase/master/replication/TransitPeerSyncReplicationStateProcedure.java
| 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 |
org/apache/hadoop/hbase/master/snapshot/EnabledTableSnapshotHandler.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 78 |
org/apache/hadoop/hbase/master/snapshot/SnapshotManager.java
| 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 |
org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 306 |
org/apache/hadoop/hbase/metrics/impl/TestGauge.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SimpleGauge should be declared as final. | 46 |
org/apache/hadoop/hbase/mob/CachedMobFile.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 58 |
org/apache/hadoop/hbase/mob/DefaultMobStoreCompactor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method performCompaction length is 309 lines (max allowed is 150). | 300 |
org/apache/hadoop/hbase/mob/DefaultMobStoreFlusher.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 114 |
org/apache/hadoop/hbase/mob/MobTestUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 46 |
org/apache/hadoop/hbase/mob/RSMobFileCleanerChore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method chore length is 179 lines (max allowed is 150). | 82 |
org/apache/hadoop/hbase/mob/mapreduce/MobRefReporter.java
| 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 |
org/apache/hadoop/hbase/monitoring/MemoryBoundedLogMessageBuffer.java
| 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 |
org/apache/hadoop/hbase/monitoring/MonitoredRPCHandlerImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 124 |
org/apache/hadoop/hbase/monitoring/TaskMonitor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MonitorRunnable should be declared as final. | 318 |
org/apache/hadoop/hbase/mttr/IntegrationTestMTTR.java
| 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 |
org/apache/hadoop/hbase/namequeues/LogHandlerUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 32 |
org/apache/hadoop/hbase/namequeues/NamedQueueRecorder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NamedQueueRecorder should be declared as final. | 40 |
org/apache/hadoop/hbase/nio/ByteBuff.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | UpperEll | Should use uppercase 'L'. | 473 | |
| misc | UpperEll | Should use uppercase 'L'. | 473 |
org/apache/hadoop/hbase/nio/MultiByteBuff.java
| 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 |
org/apache/hadoop/hbase/procedure/ProcedureManagerHost.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 56 |
org/apache/hadoop/hbase/procedure/SimpleRSProcedureManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 155 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 186 |
org/apache/hadoop/hbase/procedure/TestProcedureCoordinator.java
| 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 |
org/apache/hadoop/hbase/procedure/TestProcedureDescriber.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/procedure/ZKProcedureCoordinator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 193 |
org/apache/hadoop/hbase/procedure/ZKProcedureUtil.java
| 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 |
org/apache/hadoop/hbase/procedure/flush/MasterFlushTableProcedureManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 147 |
org/apache/hadoop/hbase/procedure/flush/RegionServerFlushTableProcedureManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 295 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 305 |
org/apache/hadoop/hbase/procedure2/TestChildProcedures.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 220 |
org/apache/hadoop/hbase/procedure2/TestProcedureBypass.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 64 |
org/apache/hadoop/hbase/procedure2/TestProcedureEvents.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 210 |
org/apache/hadoop/hbase/procedure2/TestProcedureExecutor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 192 |
org/apache/hadoop/hbase/procedure2/TestProcedureInMemoryChore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 116 |
org/apache/hadoop/hbase/procedure2/TestProcedureMetrics.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 169 |
org/apache/hadoop/hbase/procedure2/TestProcedureNonce.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 281 |
org/apache/hadoop/hbase/procedure2/TestProcedureRecovery.java
| 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 |
org/apache/hadoop/hbase/procedure2/TestProcedureReplayOrder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcedureEnv should be declared as final. | 165 |
org/apache/hadoop/hbase/procedure2/TestProcedureSuspended.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestProcEnv should be declared as final. | 270 |
org/apache/hadoop/hbase/procedure2/TestYieldProcedures.java
| 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 |
org/apache/hadoop/hbase/protobuf/ProtobufMagic.java
| 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 |
org/apache/hadoop/hbase/protobuf/ProtobufMessageConverter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ProtobufMessageConverter should be declared as final. | 51 |
org/apache/hadoop/hbase/protobuf/ReplicationProtobufUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 45 |
org/apache/hadoop/hbase/quotas/GlobalQuotaSettingsImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method merge length is 175 lines (max allowed is 150). | 187 |
org/apache/hadoop/hbase/quotas/MasterQuotaManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NamedLock should be declared as final. | 625 |
org/apache/hadoop/hbase/quotas/NoopOperationQuota.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoopOperationQuota should be declared as final. | 32 |
org/apache/hadoop/hbase/quotas/NoopQuotaLimiter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoopQuotaLimiter should be declared as final. | 26 |
org/apache/hadoop/hbase/quotas/QuotaLimiterFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 26 |
org/apache/hadoop/hbase/quotas/QuotaSettingsFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 33 |
org/apache/hadoop/hbase/quotas/QuotaTableUtil.java
| 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 |
org/apache/hadoop/hbase/quotas/SpaceQuotaHelperForTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 462 |
org/apache/hadoop/hbase/quotas/SpaceQuotaSnapshotNotifierFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SpaceQuotaSnapshotNotifierFactory should be declared as final. | 28 |
org/apache/hadoop/hbase/quotas/SpaceViolationPolicyEnforcementFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SpaceViolationPolicyEnforcementFactory should be declared as final. | 35 |
org/apache/hadoop/hbase/quotas/TestQuotaSettingsFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 106). | 56 |
org/apache/hadoop/hbase/quotas/TimeBasedLimiter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TimeBasedLimiter should be declared as final. | 31 |
org/apache/hadoop/hbase/quotas/UserQuotaState.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 190 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 194 |
org/apache/hadoop/hbase/quotas/policies/BaseViolationPolicyEnforcement.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 26 |
org/apache/hadoop/hbase/regionserver/AbstractMemStore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 233 |
org/apache/hadoop/hbase/regionserver/CellArrayMap.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 51 |
org/apache/hadoop/hbase/regionserver/ChunkCreator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 413 |
org/apache/hadoop/hbase/regionserver/CompactedHFilesDischarger.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 86 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 88 |
org/apache/hadoop/hbase/regionserver/CompactingMemStore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 591 | |
| design | FinalClass | Class InMemoryCompactionRunnable should be declared as final. | 610 |
org/apache/hadoop/hbase/regionserver/CompactionTool.java
| 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 |
org/apache/hadoop/hbase/regionserver/CompositeImmutableSegment.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 66 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 125 |
org/apache/hadoop/hbase/regionserver/CreateRandomStoreFile.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 169 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 284 |
org/apache/hadoop/hbase/regionserver/DataTieringManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DataTieringManager should be declared as final. | 42 |
org/apache/hadoop/hbase/regionserver/DefaultStoreEngine.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DefaultCompactionContext should be declared as final. | 113 |
org/apache/hadoop/hbase/regionserver/DefaultStoreFlusher.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 51 |
org/apache/hadoop/hbase/regionserver/FlushPolicyFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 35 |
org/apache/hadoop/hbase/regionserver/HRegion.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method doClose length is 274 lines (max allowed is 150). | 1759 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2084 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2091 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 2573 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 2971 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3040 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 3143 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 5007 | |
| sizes | MethodLength | Method replayRecoveredEdits length is 201 lines (max allowed is 150). | 5743 | |
| sizes | MethodLength | Method bulkLoadHFiles length is 214 lines (max allowed is 150). | 7423 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 8740 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 8741 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 8743 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 8744 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 8756 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 8757 |
org/apache/hadoop/hbase/regionserver/HRegionFileSystem.java
| 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 |
org/apache/hadoop/hbase/regionserver/HRegionServer.java
| 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 |
org/apache/hadoop/hbase/regionserver/HRegionServerCommandLine.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 82 |
org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FlushHandler should be declared as final. | 309 |
org/apache/hadoop/hbase/regionserver/MetricsRegionServerSource.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 113). | 307 |
org/apache/hadoop/hbase/regionserver/MetricsTableWrapperAggregateImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MetricsTableValues should be declared as final. | 397 |
org/apache/hadoop/hbase/regionserver/MetricsUserAggregateFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MetricsUserAggregateFactory should be declared as final. | 23 |
org/apache/hadoop/hbase/regionserver/OnlineRegions.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 45 |
org/apache/hadoop/hbase/regionserver/RSAnnotationReadingPriorityFunction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 133 |
org/apache/hadoop/hbase/regionserver/RSRpcServices.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 583 | |
| sizes | MethodLength | Method doNonAtomicRegionMutation length is 161 lines (max allowed is 150). | 748 | |
| sizes | LineLength | Line is longer than 100 characters (found 102). | 1261 | |
| sizes | MethodLength | Method multi length is 236 lines (max allowed is 150). | 2691 | |
| sizes | MethodLength | Method scan length is 228 lines (max allowed is 150). | 3341 | |
| sizes | MethodLength | Method scan length is 263 lines (max allowed is 150). | 3578 |
org/apache/hadoop/hbase/regionserver/RSSnapshotVerifier.java
| 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 |
org/apache/hadoop/hbase/regionserver/RegionAsTable.java
| 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 |
org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 1400 |
org/apache/hadoop/hbase/regionserver/RegionScannerImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method nextInternal length is 217 lines (max allowed is 150). | 419 |
org/apache/hadoop/hbase/regionserver/RegionServerServices.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 239 |
org/apache/hadoop/hbase/regionserver/RegionSplitPolicy.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 144). | 126 |
org/apache/hadoop/hbase/regionserver/ScanOptions.java
| 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 |
org/apache/hadoop/hbase/regionserver/ServerNonceManager.java
| 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 |
org/apache/hadoop/hbase/regionserver/ShutdownHook.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 39 |
org/apache/hadoop/hbase/regionserver/StatefulStoreMockMaker.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CancelAnswer should be declared as final. | 48 |
org/apache/hadoop/hbase/regionserver/StoreFileComparators.java
| 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 |
org/apache/hadoop/hbase/regionserver/StoreFileReader.java
| 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 |
org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
| 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 |
org/apache/hadoop/hbase/regionserver/StoreFileWriter.java
| 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 |
org/apache/hadoop/hbase/regionserver/StoreScanner.java
| 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 |
org/apache/hadoop/hbase/regionserver/StorefileRefresherChore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 96 |
org/apache/hadoop/hbase/regionserver/StripeMultiFileWriter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 199 |
org/apache/hadoop/hbase/regionserver/StripeStoreEngine.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class StripeCompaction should be declared as final. | 72 |
org/apache/hadoop/hbase/regionserver/StripeStoreFileManager.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestAtomicOperation.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestBlocksScanned.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 118 | |
| coding | EmptyStatement | Empty statement. | 119 |
org/apache/hadoop/hbase/regionserver/TestBulkloadBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 179 |
org/apache/hadoop/hbase/regionserver/TestBytesReadServerSideScanMetrics.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MyNoOpEncodedSeeker should be declared as final. | 887 |
org/apache/hadoop/hbase/regionserver/TestCellFlatSet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 296 |
org/apache/hadoop/hbase/regionserver/TestCellSkipListSet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 165 |
org/apache/hadoop/hbase/regionserver/TestClearRegionBlockCache.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'while' construct must use '{}'s. | 180 | |
| coding | EmptyStatement | Empty statement. | 181 |
org/apache/hadoop/hbase/regionserver/TestColumnSeeking.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestCompactingMemStore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class EnvironmentEdgeForMemstoreTest should be declared as final. | 899 |
org/apache/hadoop/hbase/regionserver/TestCompaction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 680 |
org/apache/hadoop/hbase/regionserver/TestCompactionState.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 216 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 217 |
org/apache/hadoop/hbase/regionserver/TestCompoundBloomFilter.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestCustomCellDataTieringManager.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestDataTieringManager.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestDefaultMemStore.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestFSErrorsExposed.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 231 |
org/apache/hadoop/hbase/regionserver/TestFailedAppendAndSync.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 296 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 304 |
org/apache/hadoop/hbase/regionserver/TestHRegion.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 4471 | |
| design | FinalClass | Class PutThread should be declared as final. | 4695 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 4867 | |
| blocks | EmptyBlock | Must have at least one statement. | 4930 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 5081 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 5092 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 5093 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 5111 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 5800 |
org/apache/hadoop/hbase/regionserver/TestHRegionFileSystem.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestHRegionOnCluster.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 151 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 166 |
org/apache/hadoop/hbase/regionserver/TestHRegionReplayEvents.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestHRegionServerBulkLoad.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestHStoreFile.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method checkHalfHFile length is 161 lines (max allowed is 150). | 485 |
org/apache/hadoop/hbase/regionserver/TestKeyValueScanFixture.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 46 |
org/apache/hadoop/hbase/regionserver/TestMajorCompaction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 479 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 498 |
org/apache/hadoop/hbase/regionserver/TestMetricsRegionServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 114). | 358 |
org/apache/hadoop/hbase/regionserver/TestMultiColumnScanner.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestMultiVersionConcurrencyControl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 65 |
org/apache/hadoop/hbase/regionserver/TestRSChoresScheduled.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestChoreField should be declared as final. | 60 |
org/apache/hadoop/hbase/regionserver/TestRSKilledWhenInitializing.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 135 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 178 |
org/apache/hadoop/hbase/regionserver/TestRegionIncrement.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestRegionReplicasWithRestartScenarios.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/regionserver/TestRegionServerReportForDuty.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 24 |
org/apache/hadoop/hbase/regionserver/TestResettingCounters.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestReversibleScanners.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestScanWithBloomError.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 153 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 181 |
org/apache/hadoop/hbase/regionserver/TestScannerHeartbeatMessages.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestScannerRPCScanMetrics.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 125 |
org/apache/hadoop/hbase/regionserver/TestScannerRetriableFailure.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 170 |
org/apache/hadoop/hbase/regionserver/TestSecureBulkloadListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 259 |
org/apache/hadoop/hbase/regionserver/TestSeekOptimizations.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 331 | |
| blocks | EmptyBlock | Must have at least one statement. | 441 |
org/apache/hadoop/hbase/regionserver/TestServerNonceManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 236 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 260 |
org/apache/hadoop/hbase/regionserver/TestSplitTransactionOnCluster.java
| 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 |
org/apache/hadoop/hbase/regionserver/TestTags.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 320 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 631 |
org/apache/hadoop/hbase/regionserver/TestWalAndCompactingMemStoreFlush.java
| 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 |
org/apache/hadoop/hbase/regionserver/compactions/CompactionProgress.java
| 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 |
org/apache/hadoop/hbase/regionserver/compactions/Compactor.java
| 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 |
org/apache/hadoop/hbase/regionserver/compactions/CurrentHourProvider.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CurrentHourProvider should be declared as final. | 26 |
org/apache/hadoop/hbase/regionserver/compactions/CustomCellTieredUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CustomCellTieredUtils should be declared as final. | 29 |
org/apache/hadoop/hbase/regionserver/compactions/StripeCompactionPolicy.java
| 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 |
org/apache/hadoop/hbase/regionserver/compactions/TestCompactor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 57 |
org/apache/hadoop/hbase/regionserver/compactions/TestCustomCellTieredCompactor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testCustomCellTieredCompactorWithRowKeyDateTieringValue length is 169 lines (max allowed is 150). | 154 |
org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 202 |
org/apache/hadoop/hbase/regionserver/metrics/TestMetricsThrottleExceptions.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 132 |
org/apache/hadoop/hbase/regionserver/querymatcher/NewVersionBehaviorTracker.java
| 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 |
org/apache/hadoop/hbase/regionserver/querymatcher/TestScanDeleteTracker.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/regionserver/querymatcher/TestUserScanQueryMatcher.java
| 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 |
org/apache/hadoop/hbase/regionserver/snapshot/RegionServerSnapshotManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 356 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 365 |
org/apache/hadoop/hbase/regionserver/wal/AbstractProtobufLogWriter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 247 |
org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 151 |
org/apache/hadoop/hbase/regionserver/wal/Compressor.java
| 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 |
org/apache/hadoop/hbase/regionserver/wal/FSHLog.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 109). | 184 |
org/apache/hadoop/hbase/regionserver/wal/TestLogRolling.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 461 |
org/apache/hadoop/hbase/regionserver/wal/TestLogRollingNoCluster.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 184 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 185 |
org/apache/hadoop/hbase/regionserver/wal/WALCoprocessorHost.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WALEnvironment should be declared as final. | 50 |
org/apache/hadoop/hbase/regionserver/wal/WALUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WALUtil should be declared as final. | 59 |
org/apache/hadoop/hbase/replication/ReplicationLoadSink.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ReplicationLoadSink should be declared as final. | 25 |
org/apache/hadoop/hbase/replication/ReplicationPeerConfig.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ReplicationPeerConfig should be declared as final. | 37 |
org/apache/hadoop/hbase/replication/TestHBaseReplicationEndpoint.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DummyHBaseReplicationEndpoint should be declared as final. | 188 |
org/apache/hadoop/hbase/replication/TestRemovePeerProcedureWaitForSCP.java
| 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 |
org/apache/hadoop/hbase/replication/master/ReplicationLogCleaner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 101). | 81 |
org/apache/hadoop/hbase/replication/regionserver/ReplicationLoad.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 127 |
org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceInterface.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 107). | 205 |
org/apache/hadoop/hbase/replication/regionserver/ReplicationSyncUp.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 201 |
org/apache/hadoop/hbase/replication/regionserver/TestReplicationSink.java
| 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 |
org/apache/hadoop/hbase/rest/RESTServer.java
| 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 |
org/apache/hadoop/hbase/rest/RESTServlet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 148 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 149 |
org/apache/hadoop/hbase/rest/RowResource.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 289 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 289 |
org/apache/hadoop/hbase/rest/RowResourceBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 472 |
org/apache/hadoop/hbase/rest/RowSpec.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 104 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 121 |
org/apache/hadoop/hbase/rest/TableResource.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 283). | 151 |
org/apache/hadoop/hbase/rest/TestSecureRESTServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class EmptyCredentials should be declared as final. | 441 |
org/apache/hadoop/hbase/rest/client/Client.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 443 |
org/apache/hadoop/hbase/rest/model/ScannerModel.java
| 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 |
org/apache/hadoop/hbase/rest/model/TableRegionModel.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 124 |
org/apache/hadoop/hbase/rest/model/TestScannerModel.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 117). | 55 | |
| sizes | LineLength | Line is longer than 100 characters (found 112). | 56 |
org/apache/hadoop/hbase/rest/model/VersionModel.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 79 |
org/apache/hadoop/hbase/security/AbstractTestSecureIPC.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 328 |
org/apache/hadoop/hbase/security/AuthMethod.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'authenticationMethod' must be private and have accessor methods. | 36 |
org/apache/hadoop/hbase/security/HBaseSaslRpcClient.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 217 |
org/apache/hadoop/hbase/security/SaslUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 37 |
org/apache/hadoop/hbase/security/TestUser.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/security/User.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 299 |
org/apache/hadoop/hbase/security/access/AccessControlClient.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 40 |
org/apache/hadoop/hbase/security/access/AccessControlFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 166 |
org/apache/hadoop/hbase/security/access/AccessControlUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class AccessControlUtil should be declared as final. | 45 |
org/apache/hadoop/hbase/security/access/AccessController.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method checkCoveringPermission length is 169 lines (max allowed is 150). | 437 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 551 |
org/apache/hadoop/hbase/security/access/AuthManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class PermissionCache should be declared as final. | 63 |
org/apache/hadoop/hbase/security/access/SecureTestUtil.java
| 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 |
org/apache/hadoop/hbase/security/access/ShadedAccessControlUtil.java
| 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 |
org/apache/hadoop/hbase/security/access/TestAccessController.java
| 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 |
org/apache/hadoop/hbase/security/access/TestAccessController2.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 304 | |
| blocks | EmptyBlock | Must have at least one statement. | 330 |
org/apache/hadoop/hbase/security/access/TestCellACLs.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testCellPermissions length is 220 lines (max allowed is 150). | 151 |
org/apache/hadoop/hbase/security/access/TestHDFSAclHelper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.apache.hadoop.hbase.client.*. | 32 |
org/apache/hadoop/hbase/security/access/TestNamespaceCommands.java
| 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 |
org/apache/hadoop/hbase/security/access/TestWithDisabledAuthorization.java
| 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 |
org/apache/hadoop/hbase/security/access/ZKPermissionWatcher.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 232 |
org/apache/hadoop/hbase/security/provider/CustomSaslAuthenticationProviderTestBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class InMemoryServerProviderCallbackHandler should be declared as final. | 295 |
org/apache/hadoop/hbase/security/provider/GssSaslServerAuthenticationProvider.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class SaslGssCallbackHandler should be declared as final. | 73 |
org/apache/hadoop/hbase/security/token/SecureTestCluster.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 36 |
org/apache/hadoop/hbase/security/token/TestZKSecretWatcher.java
| 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 |
org/apache/hadoop/hbase/security/token/TestZKSecretWatcherRefreshKeys.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockAbortable should be declared as final. | 54 |
org/apache/hadoop/hbase/security/token/TokenUtil.java
| 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 |
org/apache/hadoop/hbase/security/visibility/CellVisibility.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 63 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 64 |
org/apache/hadoop/hbase/security/visibility/DefaultVisibilityLabelServiceImpl.java
| 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 |
org/apache/hadoop/hbase/security/visibility/ExpAsStringVisibilityLabelServiceImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 433 |
org/apache/hadoop/hbase/security/visibility/ExpressionParser.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 76 |
org/apache/hadoop/hbase/security/visibility/LabelFilteringScanLabelGenerator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 45 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 48 |
org/apache/hadoop/hbase/security/visibility/TestExpressionExpander.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testPositiveCases length is 359 lines (max allowed is 150). | 41 |
org/apache/hadoop/hbase/security/visibility/TestExpressionParser.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testPositiveCases length is 253 lines (max allowed is 150). | 44 |
org/apache/hadoop/hbase/security/visibility/TestVisibilityLabelsReplication.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 122 |
org/apache/hadoop/hbase/security/visibility/TestVisibilityLabelsWithDefaultVisLabelService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'else' construct must use '{}'s. | 148 |
org/apache/hadoop/hbase/security/visibility/TestVisibilityLabelsWithDeletes.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testDeleteColumnLatestWithNoCellVisibility length is 155 lines (max allowed is 150). | 2582 |
org/apache/hadoop/hbase/security/visibility/TestVisibilityLablesWithGroups.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method testGroupAuths length is 239 lines (max allowed is 150). | 119 |
org/apache/hadoop/hbase/security/visibility/VisibilityClient.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 52 |
org/apache/hadoop/hbase/security/visibility/VisibilityConstants.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
org/apache/hadoop/hbase/security/visibility/VisibilityController.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 582 |
org/apache/hadoop/hbase/security/visibility/VisibilityLabelServiceManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class VisibilityLabelServiceManager should be declared as final. | 30 |
org/apache/hadoop/hbase/security/visibility/VisibilityLabelsCache.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class VisibilityLabelsCache should be declared as final. | 48 |
org/apache/hadoop/hbase/security/visibility/VisibilityLabelsValidator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 26 |
org/apache/hadoop/hbase/security/visibility/VisibilityNewVersionBehaivorTracker.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TagInfo should be declared as final. | 55 |
org/apache/hadoop/hbase/security/visibility/VisibilityTestUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 |
org/apache/hadoop/hbase/security/visibility/VisibilityUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 72 |
org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java
| 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 |
org/apache/hadoop/hbase/shaded/protobuf/ResponseConverter.java
| 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 |
org/apache/hadoop/hbase/slowlog/SlowLogTableAccessor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 44 |
org/apache/hadoop/hbase/snapshot/ExportSnapshot.java
| 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 |
org/apache/hadoop/hbase/snapshot/MobSnapshotTestingUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 44 |
org/apache/hadoop/hbase/snapshot/RestoreSnapshotHelper.java
| 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 |
org/apache/hadoop/hbase/snapshot/SnapshotDescriptionUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 473 |
org/apache/hadoop/hbase/snapshot/SnapshotInfo.java
| 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 |
org/apache/hadoop/hbase/snapshot/SnapshotManifest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 348 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 478 |
org/apache/hadoop/hbase/snapshot/SnapshotManifestV2.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 146 |
org/apache/hadoop/hbase/snapshot/SnapshotTestingUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 556 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 589 |
org/apache/hadoop/hbase/test/IntegrationTestBigLinkedList.java
| 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 |
org/apache/hadoop/hbase/test/IntegrationTestBigLinkedListWithVisibility.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 337 |
org/apache/hadoop/hbase/test/IntegrationTestLoadAndVerify.java
| 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 |
org/apache/hadoop/hbase/test/IntegrationTestTimeBoundedMultiGetRequestsWithRegionReplicas.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 66 |
org/apache/hadoop/hbase/test/IntegrationTestTimeBoundedRequestsWithRegionReplicas.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 227 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 347 |
org/apache/hadoop/hbase/test/IntegrationTestWithCellVisibilityLoadAndVerify.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 401 |
org/apache/hadoop/hbase/test/util/warc/WARCInputFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WARCReader should be declared as final. | 79 |
org/apache/hadoop/hbase/testclassification/ClientTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 38 |
org/apache/hadoop/hbase/testclassification/CoprocessorTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/FilterTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/FlakeyTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/IOTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 38 |
org/apache/hadoop/hbase/testclassification/IntegrationTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 36 |
org/apache/hadoop/hbase/testclassification/LargeTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 35 |
org/apache/hadoop/hbase/testclassification/MapReduceTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/MasterTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/MediumTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 34 |
org/apache/hadoop/hbase/testclassification/MetricsTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 23 |
org/apache/hadoop/hbase/testclassification/MiscTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/RPCTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/RSGroupTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 23 |
org/apache/hadoop/hbase/testclassification/RegionServerTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/ReplicationTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/RestTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/SecurityTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 37 |
org/apache/hadoop/hbase/testclassification/SmallTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 32 |
org/apache/hadoop/hbase/testclassification/VerySlowMapReduceTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 38 |
org/apache/hadoop/hbase/testclassification/VerySlowRegionServerTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 38 |
org/apache/hadoop/hbase/testclassification/ZKTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 24 |
org/apache/hadoop/hbase/thrift/TestThriftServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 732 |
org/apache/hadoop/hbase/thrift2/DemoClient.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 100 characters (found 103). | 110 |
org/apache/hadoop/hbase/tool/MapreduceTestingShim.java
| 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 |
org/apache/hadoop/hbase/tool/coprocessor/CoprocessorValidatorTest.java
| 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 |
org/apache/hadoop/hbase/trace/OpenTelemetryClassRule.java
| 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 |
org/apache/hadoop/hbase/types/TestStruct.java
| 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 |
org/apache/hadoop/hbase/util/AbstractHBaseTool.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 281 |
org/apache/hadoop/hbase/util/AbstractHBaseToolTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestTool should be declared as final. | 48 |
org/apache/hadoop/hbase/util/Addressing.java
| 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 |
org/apache/hadoop/hbase/util/AvlUtil.java
| 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 |
org/apache/hadoop/hbase/util/BoundedPriorityBlockingQueue.java
| 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 |
org/apache/hadoop/hbase/util/Bytes.java
| 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 |
org/apache/hadoop/hbase/util/ClassSize.java
| 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 |
org/apache/hadoop/hbase/util/Classes.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
org/apache/hadoop/hbase/util/CompressionTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 53 |
org/apache/hadoop/hbase/util/ConcurrentMapUtils.java
| 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 |
org/apache/hadoop/hbase/util/ConnectionCache.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 129 |
org/apache/hadoop/hbase/util/CoprocessorClassLoader.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class CoprocessorClassLoader should be declared as final. | 71 |
org/apache/hadoop/hbase/util/DirectMemoryUtils.java
| 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 |
org/apache/hadoop/hbase/util/EncryptionTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class EncryptionTest should be declared as final. | 36 |
org/apache/hadoop/hbase/util/EnvironmentEdgeManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class EnvironmentEdgeManager should be declared as final. | 56 |
org/apache/hadoop/hbase/util/FSUtils.java
| 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 |
org/apache/hadoop/hbase/util/GetJavaProperty.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 27 |
org/apache/hadoop/hbase/util/HBaseConfTool.java
| 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 |
org/apache/hadoop/hbase/util/HBaseFsck.java
| 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 |
org/apache/hadoop/hbase/util/HBaseFsckRepair.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 54 |
org/apache/hadoop/hbase/util/HBaseHomePath.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class HBaseHomePath should be declared as final. | 24 |
org/apache/hadoop/hbase/util/HFileArchiveTestingUtil.java
| 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 |
org/apache/hadoop/hbase/util/HFileArchiveUtil.java
| 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 |
org/apache/hadoop/hbase/util/HFileTestUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 52 |
org/apache/hadoop/hbase/util/HashedBytes.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 51 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 52 |
org/apache/hadoop/hbase/util/JSONBean.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | ArrayTypeStyle | Array brackets at illegal position. | 255 |
org/apache/hadoop/hbase/util/JVM.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 181 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 305 |
org/apache/hadoop/hbase/util/JVMClusterUtil.java
| 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 |
org/apache/hadoop/hbase/util/JenkinsHash.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | InnerAssignment | Inner assignments should be avoided. | 104 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 104 |
org/apache/hadoop/hbase/util/JvmPauseMonitor.java
| 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 |
org/apache/hadoop/hbase/util/KerberosUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 |
org/apache/hadoop/hbase/util/KeyLocker.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 64 |
org/apache/hadoop/hbase/util/LoadTestDataGeneratorWithMOB.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
org/apache/hadoop/hbase/util/LoadTestTool.java
| 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 |
org/apache/hadoop/hbase/util/LoadTestUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 46 |
org/apache/hadoop/hbase/util/LogMonitoring.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 68 |
org/apache/hadoop/hbase/util/MD5Hash.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 |
org/apache/hadoop/hbase/util/ModifyRegionUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 97 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 129 |
org/apache/hadoop/hbase/util/MultiThreadedAction.java
| 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 |
org/apache/hadoop/hbase/util/MultiThreadedWriterBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WroteKeysTracker should be declared as final. | 123 |
org/apache/hadoop/hbase/util/OrderedBytes.java
| 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 |
org/apache/hadoop/hbase/util/PairOfSameType.java
| 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 |
org/apache/hadoop/hbase/util/PrettyPrinter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 32 |
org/apache/hadoop/hbase/util/ReflectionUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 41 |
org/apache/hadoop/hbase/util/RegionMover.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class RegionMover should be declared as final. | 93 |
org/apache/hadoop/hbase/util/RegionSplitCalculator.java
| 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 |
org/apache/hadoop/hbase/util/RegionSplitter.java
| 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 |
org/apache/hadoop/hbase/util/ServerCommandLine.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 123 | |
| misc | ArrayTypeStyle | Array brackets at illegal position. | 138 |
org/apache/hadoop/hbase/util/ShutdownHookManager.java
| 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 |
org/apache/hadoop/hbase/util/SimpleByteRange.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | EqualsHashCode | Definition of 'equals()' without corresponding definition of 'hashCode()'. | 125 |
org/apache/hadoop/hbase/util/SimpleMutableByteRange.java
| 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 |
org/apache/hadoop/hbase/util/SimplePositionedMutableByteRange.java
| 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 |
org/apache/hadoop/hbase/util/StealJobQueue.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 122 |
org/apache/hadoop/hbase/util/TestAvlUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestAvlKeyComparator should be declared as final. | 264 |
org/apache/hadoop/hbase/util/TestBloomFilterChunk.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 121 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 163 |
org/apache/hadoop/hbase/util/TestCompressionTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/util/TestFSUtils.java
| 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 |
org/apache/hadoop/hbase/util/TestIdLock.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 101 |
org/apache/hadoop/hbase/util/TestIdReadWriteLockWithObjectPool.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'for' construct must use '{}'s. | 126 |
org/apache/hadoop/hbase/util/TestRecoverLeaseFSUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class IsFileClosedDistributedFileSystem should be declared as final. | 142 |
org/apache/hadoop/hbase/util/TestRegionMover3.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockRackManager should be declared as final. | 160 |
org/apache/hadoop/hbase/util/TestRegionMoverWithRSGroupEnable.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 96 |
org/apache/hadoop/hbase/util/TestServerHttpUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class TestServerHttpUtils should be declared as final. | 29 |
org/apache/hadoop/hbase/util/TestSortedList.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.junit.Assert.*. | 20 |
org/apache/hadoop/hbase/util/Threads.java
| 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 |
org/apache/hadoop/hbase/util/Triple.java
| 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 |
org/apache/hadoop/hbase/util/VersionInfo.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 35 |
org/apache/hadoop/hbase/util/Writables.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 34 |
org/apache/hadoop/hbase/util/bulkdatagenerator/BulkDataGeneratorInputFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class FakeInputSplit should be declared as final. | 67 |
org/apache/hadoop/hbase/util/hbck/HbckTestingUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 35 |
org/apache/hadoop/hbase/wal/BoundedGroupingStrategy.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 58 |
org/apache/hadoop/hbase/wal/DisabledWALProvider.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 223 |
org/apache/hadoop/hbase/wal/IOTestProvider.java
| 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 |
org/apache/hadoop/hbase/wal/TestWALFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 448 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 506 |
org/apache/hadoop/hbase/wal/TestWALRootDir.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 132 |
org/apache/hadoop/hbase/wal/TestWALSplit.java
| 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 |
org/apache/hadoop/hbase/wal/WALPerformanceEvaluation.java
| 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 |
org/apache/hadoop/hbase/wal/WALSplitter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method splitWAL length is 154 lines (max allowed is 150). | 280 |
org/apache/hadoop/hbase/zookeeper/TestZKLeaderManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class MockAbortable should be declared as final. | 56 |
org/apache/hadoop/hbase/zookeeper/TestZKMainServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class NoExitSecurityManager should be declared as final. | 52 |
org/apache/hadoop/hbase/zookeeper/TestZKMulti.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ZKMultiAbortable should be declared as final. | 62 |
org/apache/hadoop/hbase/zookeeper/TestZKUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class WarnOnlyAbortable should be declared as final. | 233 |
org/apache/hadoop/hbase/zookeeper/TestZooKeeperACL.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class DummySecurityConfiguration should be declared as final. | 310 |
org/apache/hadoop/hbase/zookeeper/ZKMetadata.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ZKMetadata should be declared as final. | 27 |
org/apache/hadoop/hbase/zookeeper/ZNodePaths.java
| 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 |
