Checkstyle Results
The following document contains the results of Checkstyle 8.29 with hbase/checkstyle.xml ruleset. 
Summary
| Files |  |
 |
 |
|---|---|---|---|
| 4862 | 0 | 0 | 1732 |
Files
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| annotation | MissingDeprecated | 10 | |
| blocks | EmptyBlock | 30 | |
| NeedBraces | 939 | | |
| coding | EmptyStatement | 19 | |
| EqualsHashCode | 2 | | |
| InnerAssignment | 58 | | |
| design | FinalClass | 53 | |
| HideUtilityClassConstructor | 81 | | |
| InterfaceIsType | 1 | | |
VisibilityModifier
|
101 | | |
| imports | AvoidStarImport | 22 | |
| indentation | Indentation
|
35 | |
| javadoc | NonEmptyAtclauseDescription | 49 | |
| misc | ArrayTypeStyle | 113 | |
| UpperEll | 23 | | |
| sizes | LineLength
|
134 | |
| 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 |
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. | 312 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 315 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 324 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 333 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 360 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 366 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 393 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 504 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 611 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 682 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 815 |
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 |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 488 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 490 |
org/apache/hadoop/hbase/HBaseConfiguration.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 41 |
org/apache/hadoop/hbase/HBaseTestingUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 361 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 363 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 499 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 503 |
|
blocks | EmptyBlock | Must have at least one statement. | 1014 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1229 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2024 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 3080 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3495 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3497 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3528 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3568 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3570 |
org/apache/hadoop/hbase/HBaseTestingUtility.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 349 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 351 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 487 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 491 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 502 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 587 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 602 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 618 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 618 |
|
blocks | EmptyBlock | Must have at least one statement. | 1247 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1462 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2269 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 3394 |
|
blocks | EmptyBlock | Must have at least one statement. | 3519 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3808 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3810 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3843 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3884 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3886 |
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 |
|---|---|---|---|---|
|
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/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/KeyValueTestUtil.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. | 66 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 68 |
org/apache/hadoop/hbase/KeyValueUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 45 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 420 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 712 |
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. | 59 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 624 |
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. | 156 |
org/apache/hadoop/hbase/PerformanceEvaluation.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 440 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 492 |
|
sizes | LineLength | Line is longer than 100 characters (found 101). | 582 |
|
sizes | LineLength | Line is longer than 100 characters (found 102). | 583 |
|
misc | UpperEll | Should use uppercase 'L'. | 730 |
|
misc | UpperEll | Should use uppercase 'L'. | 730 |
|
misc | UpperEll | Should use uppercase 'L'. | 730 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1199 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 1200 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1209 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1216 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1225 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1230 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1387 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1389 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1550 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 1812 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 1860 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1969 |
|
misc | UpperEll | Should use uppercase 'L'. | 2151 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 2315 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 2399 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2556 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2557 |
|
sizes | LineLength | Line is longer than 100 characters (found 101). | 2667 |
|
sizes | MethodLength | Method length is 300 lines (max allowed is 150). | 2701 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2865 |
org/apache/hadoop/hbase/PerformanceEvaluationCommons.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 30 |
org/apache/hadoop/hbase/PrivateCellUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2485 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2489 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2492 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2496 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 2497 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2627 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2785 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2798 |
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. | 515 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 516 |
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/TestClientClusterStatus.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 176 |
org/apache/hadoop/hbase/TestGlobalMemStoreSize.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 175 |
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 |
org/apache/hadoop/hbase/TestPartialResultsFromClientSide.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 334 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 386 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 643 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 664 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 670 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 671 |
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/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/HFileArchiver.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | FinalClass | Class HFileArchiver should be declared as final. | 62 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 351 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 760 |
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. | 41 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 122 |
org/apache/hadoop/hbase/chaos/ChaosUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 29 |
org/apache/hadoop/hbase/chaos/factories/MonkeyConstants.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | InterfaceIsType | interfaces should describe a type and hence have methods. | 24 |
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 |
|---|---|---|---|---|
|
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). | 730 |
|
sizes | LineLength | Line is longer than 100 characters (found 112). | 741 |
|
sizes | LineLength | Line is longer than 100 characters (found 112). | 767 |
org/apache/hadoop/hbase/client/AsyncAdmin.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 1369 |
org/apache/hadoop/hbase/client/AsyncBatchRpcRetryingCaller.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | VisibilityModifier | Variable 'loc' must be private and have accessor methods. | 123 |
|
design | VisibilityModifier | Variable 'actions' must be private and have accessor methods. | 125 |
|
design | VisibilityModifier | Variable 'actionsByRegion' must be private and have accessor methods. | 134 |
org/apache/hadoop/hbase/client/AsyncClientScanner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | VisibilityModifier | Variable 'loc' must be private and have accessor methods. | 138 |
|
design | VisibilityModifier | Variable 'stub' must be private and have accessor methods. | 142 |
|
design | VisibilityModifier | Variable 'controller' must be private and have accessor methods. | 144 |
|
design | VisibilityModifier | Variable 'resp' must be private and have accessor methods. | 146 |
org/apache/hadoop/hbase/client/AsyncTableImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 243 |
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/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. | 354 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 373 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 383 |
org/apache/hadoop/hbase/client/PackagePrivateFieldAccessor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 31 |
org/apache/hadoop/hbase/client/Query.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 78 |
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/Result.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 838 |
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 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/TestAsyncTableScanMetrics.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 118 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 136 |
org/apache/hadoop/hbase/client/TestAsyncTableScanner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 92 |
org/apache/hadoop/hbase/client/TestAvoidCellReferencesIntoShippedBlocks.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 100 |
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 123 |
|
blocks | EmptyBlock | Must have at least one statement. | 247 |
org/apache/hadoop/hbase/client/TestBlockEvictionFromClient.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 109 |
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 130 |
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 138 |
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 148 |
org/apache/hadoop/hbase/client/TestFromClientSide3.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 111 |
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 119 |
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 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. | 97 |
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/TestRawAsyncTableScan.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 85 |
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. | 259 |
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/TestScannersFromClientSide2.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 134 |
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. | 39 |
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. | 133 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 662 |
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. | 98 |
org/apache/hadoop/hbase/coprocessor/RegionObserver.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 100 characters (found 109). | 683 |
|
sizes | LineLength | Line is longer than 100 characters (found 109). | 712 |
|
sizes | LineLength | Line is longer than 100 characters (found 109). | 841 |
|
sizes | LineLength | Line is longer than 100 characters (found 109). | 870 |
org/apache/hadoop/hbase/coprocessor/TestCoprocessorInterface.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 340 |
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. | 916 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 917 |
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. | 300 |
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. | 56 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 57 |
org/apache/hadoop/hbase/filter/FilterListWithAND.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 100 characters (found 107). | 72 |
|
sizes | LineLength | Line is longer than 100 characters (found 198). | 82 |
|
sizes | LineLength | Line is longer than 100 characters (found 198). | 83 |
|
sizes | LineLength | Line is longer than 100 characters (found 198). | 84 |
|
sizes | LineLength | Line is longer than 100 characters (found 198). | 85 |
|
sizes | LineLength | Line is longer than 100 characters (found 198). | 86 |
|
sizes | LineLength | Line is longer than 100 characters (found 198). | 87 |
|
sizes | LineLength | Line is longer than 100 characters (found 198). | 88 |
|
sizes | LineLength | Line is longer than 100 characters (found 198). | 89 |
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. | 627 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 629 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 648 |
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 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/PrefixFilter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 53 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 54 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 56 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 78 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 108 |
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 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/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. | 236 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 246 |
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/TestPrefixFilter.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/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/fs/TestBlockReorderBlockLocation.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 131 |
org/apache/hadoop/hbase/hbtop/HBTop.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 151 lines (max allowed is 150). | 61 |
org/apache/hadoop/hbase/hbtop/screen/top/TopScreenPresenter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
indentation | Indentation | 'block' child has incorrect indentation level 8, expected level should be 6. | 251 |
|
indentation | Indentation | 'block' child has incorrect indentation level 8, expected level should be 6. | 252 |
|
indentation | Indentation | 'block' child has incorrect indentation level 8, expected level should be 6. | 253 |
|
indentation | Indentation | 'block rcurly' has incorrect indentation level 6, expected level should be 4. | 254 |
|
indentation | Indentation | 'if' has incorrect indentation level 8, expected level should be 6. | 324 |
|
indentation | Indentation | 'if' child has incorrect indentation level 10, expected level should be 8. | 325 |
|
indentation | Indentation | 'if rcurly' has incorrect indentation level 8, expected level should be 6. | 326 |
|
indentation | Indentation | 'if' has incorrect indentation level 8, expected level should be 6. | 328 |
|
indentation | Indentation | 'if' child has incorrect indentation level 10, expected level should be 8. | 329 |
|
indentation | Indentation | 'if rcurly' has incorrect indentation level 8, expected level should be 6. | 330 |
|
indentation | Indentation | 'block' child has incorrect indentation level 8, expected level should be 6. | 332 |
|
indentation | Indentation | 'block' child has incorrect indentation level 8, expected level should be 6. | 333 |
|
indentation | Indentation | 'block rcurly' has incorrect indentation level 6, expected level should be 4. | 334 |
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/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/prometheus/PrometheusHadoopServlet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 73 |
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 |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 130 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 160 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 169 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 189 |
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 length is 192 lines (max allowed is 150). | 94 |
|
design | VisibilityModifier | Variable 'atEnd' must be private and have accessor methods. | 100 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 104 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 110 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 117 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 124 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 131 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 138 |
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. | 162 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 209 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 210 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 211 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 214 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 215 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 216 |
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/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). | 246 |
org/apache/hadoop/hbase/io/TestHeapSize.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 177 lines (max allowed is 150). | 116 |
|
sizes | MethodLength | Method 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/FanOutOneBlockAsyncDFSOutputHelper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 312 |
org/apache/hadoop/hbase/io/compress/Compression.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 108 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 461 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 476 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 485 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 502 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 505 |
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. | 409 |
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/X509Util.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 100 characters (found 107). | 475 |
org/apache/hadoop/hbase/io/encoding/CopyKeyDataBlockEncoder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
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 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/hfile/BlockCacheUtil.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. | 161 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 282 |
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. | 148 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 149 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 194 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 195 |
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. | 46 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 230 |
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 |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 100 characters (found 132). | 238 |
org/apache/hadoop/hbase/io/hfile/FixedFileTrailer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 100 characters (found 103). | 378 |
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. | 61 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 385 |
|
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. | 1074 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1108 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1110 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1253 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1277 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1465 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1560 |
org/apache/hadoop/hbase/io/hfile/HFilePrettyPrinter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 166 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 210 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 212 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 216 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 558 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 571 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 721 |
|
design | FinalClass | Class Builder should be declared as final. | 747 |
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/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 |
org/apache/hadoop/hbase/io/hfile/NoOpIndexBlockEncoder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | FinalClass | Class NoOpIndexBlockEncoder should be declared as final. | 43 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 87 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 255 |
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/TestCacheConfig.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'while' construct must use '{}'s. | 357 |
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. | 97 |
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. | 342 |
|
blocks | NeedBraces | 'while' construct must use '{}'s. | 351 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 368 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 638 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 689 |
org/apache/hadoop/hbase/io/hfile/TestHFileWriterV3.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 179 lines (max allowed is 150). | 124 |
org/apache/hadoop/hbase/io/hfile/TestPrefetch.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
indentation | Indentation | 'block' child has incorrect indentation level 6, expected level should be 8. | 179 |
|
indentation | Indentation | 'if' has incorrect indentation level 6, expected level should be 8. | 180 |
|
indentation | Indentation | 'if' child has incorrect indentation level 10, expected level should be 12. | 182 |
|
indentation | Indentation | 'if rparen' has incorrect indentation level 6, expected level should be 8. | 183 |
|
indentation | Indentation | 'if' child has incorrect indentation level 8, expected level should be 10. | 184 |
|
indentation | Indentation | 'if rcurly' has incorrect indentation level 6, expected level should be 8. | 185 |
|
indentation | Indentation | 'block rcurly' has incorrect indentation level 4, expected level should be 6. | 186 |
|
indentation | Indentation | 'block' child has incorrect indentation level 6, expected level should be 8. | 199 |
|
indentation | Indentation | 'if' has incorrect indentation level 6, expected level should be 8. | 200 |
|
indentation | Indentation | 'if' child has incorrect indentation level 8, expected level should be 10. | 201 |
|
indentation | Indentation | 'if rcurly' has incorrect indentation level 6, expected level should be 8. | 202 |
|
indentation | Indentation | 'if' child has incorrect indentation level 10, expected level should be 12. | 204 |
|
indentation | Indentation | 'if rparen' has incorrect indentation level 6, expected level should be 8. | 205 |
|
indentation | Indentation | 'if' child has incorrect indentation level 8, expected level should be 10. | 206 |
|
indentation | Indentation | 'if rcurly' has incorrect indentation level 6, expected level should be 8. | 207 |
|
indentation | Indentation | 'block' child has incorrect indentation level 6, expected level should be 8. | 208 |
|
indentation | Indentation | 'block rcurly' has incorrect indentation level 4, expected level should be 6. | 209 |
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. | 140 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 161 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 207 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 209 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 219 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 220 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 253 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 254 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 291 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 300 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 301 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 325 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 426 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 462 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 470 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 687 |
org/apache/hadoop/hbase/io/hfile/bucket/BucketCache.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1396 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 1400 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 1410 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1691 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1694 |
org/apache/hadoop/hbase/io/hfile/bucket/TestBucketCache.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 185 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 841 |
org/apache/hadoop/hbase/io/hfile/bucket/TestBucketWriterThread.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 103 |
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 |
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. | 34 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 137 |
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 | EmptyBlock | Must have at least one statement. | 492 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 561 |
org/apache/hadoop/hbase/ipc/BufferCallBeforeInitHandler.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | VisibilityModifier | Variable 'action' must be private and have accessor methods. | 45 |
|
design | VisibilityModifier | Variable 'error' must be private and have accessor methods. | 47 |
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. | 59 |
org/apache/hadoop/hbase/ipc/RpcServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 401 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 402 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 499 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 500 |
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 |
|---|---|---|---|---|
|
annotation | MissingDeprecated | Must include both @java.lang.Deprecated annotation and @deprecated Javadoc tag with description. | 72 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 220 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 223 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 258 |
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. | 173 |
org/apache/hadoop/hbase/ipc/TestRWQueueRpcExecutor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.mockito.Mockito.*. | 25 |
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/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/CellSortReducer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 53 |
org/apache/hadoop/hbase/mapreduce/CopyTable.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 100 characters (found 104). | 230 |
|
sizes | LineLength | Line is longer than 100 characters (found 104). | 231 |
|
sizes | LineLength | Line is longer than 100 characters (found 104). | 239 |
|
sizes | LineLength | Line is longer than 100 characters (found 108). | 242 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 405 |
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/HFileOutputFormat2.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 275 lines (max allowed is 150). | 193 |
org/apache/hadoop/hbase/mapreduce/Import.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 178 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 209 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 273 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 336 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 416 |
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. | 166 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 174 |
org/apache/hadoop/hbase/mapreduce/MapreduceTestingShim.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 104 |
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/PutCombiner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 75 |
org/apache/hadoop/hbase/mapreduce/PutSortReducer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 128 |
org/apache/hadoop/hbase/mapreduce/ResultSerialization.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 115 |
org/apache/hadoop/hbase/mapreduce/RoundRobinTableInputFormat.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
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 |
|---|---|---|---|---|
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 39 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 105 |
org/apache/hadoop/hbase/mapreduce/TableInputFormatBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 608 |
org/apache/hadoop/hbase/mapreduce/TableMapReduceUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 68 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 168 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 169 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 633 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 758 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 759 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 824 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 837 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 891 |
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 |
|---|---|---|---|---|
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 198 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 199 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 262 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 263 |
|
misc | UpperEll | Should use uppercase 'L'. | 284 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 345 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 417 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 529 |
|
sizes | MethodLength | Method length is 169 lines (max allowed is 150). | 622 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1163 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 1183 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 1185 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 1382 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 1386 |
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 |
org/apache/hadoop/hbase/mapreduce/TestRowCounter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 71 |
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 82 |
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/TestTableInputFormatScanBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 133 |
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 length is 171 lines (max allowed is 150). | 517 |
org/apache/hadoop/hbase/master/AssignmentVerificationReport.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method 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. | 832 |
|
sizes | MethodLength | Method length is 424 lines (max allowed is 150). | 932 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1961 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3518 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3520 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 3661 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3704 |
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. | 221 |
org/apache/hadoop/hbase/master/MockRegionServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 256 |
org/apache/hadoop/hbase/master/RegionPlacementMaintainer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 131 |
|
sizes | MethodLength | Method 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 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. | 542 |
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/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 |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 309 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 311 |
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/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. | 168 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 180 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 189 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 198 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 735 |
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. | 199 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 223 |
org/apache/hadoop/hbase/master/balancer/BalancerClusterState.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 232 lines (max allowed is 150). | 131 |
org/apache/hadoop/hbase/master/balancer/BalancerTestBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 336 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 413 |
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/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 length is 207 lines (max allowed is 150). | 255 |
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/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/http/MasterStatusServlet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 72 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 73 |
org/apache/hadoop/hbase/master/janitor/TestCatalogJanitor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 702 |
org/apache/hadoop/hbase/master/locking/LockManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 244 |
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 |
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. | 126 |
org/apache/hadoop/hbase/master/procedure/MasterProcedureScheduler.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 384 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 723 |
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. | 278 |
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/ProcedureDescriber.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | FinalClass | Class ProcedureDescriber should be declared as final. | 37 |
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/ServerCrashProcedure.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 160 lines (max allowed is 150). | 135 |
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/procedure/TestMasterProcedureSchedulerConcurrency.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 267 |
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 length is 158 lines (max allowed is 150). | 234 |
|
sizes | LineLength | Line is longer than 100 characters (found 112). | 289 |
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. | 260 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1187 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1244 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1248 |
org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 272 |
org/apache/hadoop/hbase/mob/CachedMobFile.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 57 |
org/apache/hadoop/hbase/mob/DefaultMobStoreCompactor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 307 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/MobFileCleanerChore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 159 lines (max allowed is 150). | 140 |
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 length is 162 lines (max allowed is 150). | 82 |
org/apache/hadoop/hbase/mob/TestMobWithByteBuffAllocator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 103 |
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/mttr/IntegrationTestMTTR.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | UpperEll | Should use uppercase 'L'. | 114 |
|
misc | UpperEll | Should use uppercase 'L'. | 330 |
org/apache/hadoop/hbase/namequeues/LogHandlerUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 31 |
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. | 138 |
org/apache/hadoop/hbase/procedure/flush/RegionServerFlushTableProcedureManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | EmptyBlock | Must have at least one statement. | 293 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 303 |
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. | 44 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 147 |
org/apache/hadoop/hbase/quotas/DefaultOperationQuota.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 80 |
org/apache/hadoop/hbase/quotas/GlobalQuotaSettingsImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 152 lines (max allowed is 150). | 167 |
org/apache/hadoop/hbase/quotas/NoopOperationQuota.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | FinalClass | Class NoopOperationQuota should be declared as final. | 29 |
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. | 25 |
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. | 518 |
org/apache/hadoop/hbase/quotas/QuotaUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 290 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 385 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 389 |
org/apache/hadoop/hbase/quotas/SpaceQuotaHelperForTests.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 472 |
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. | 32 |
org/apache/hadoop/hbase/quotas/UserQuotaState.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 57 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 193 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 197 |
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. | 236 |
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. | 83 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 85 |
org/apache/hadoop/hbase/regionserver/CompactingMemStore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 590 |
org/apache/hadoop/hbase/regionserver/CompositeImmutableSegment.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 65 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 124 |
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/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 length is 274 lines (max allowed is 150). | 1690 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2015 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2022 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2427 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 2825 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2894 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2997 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 4852 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 5200 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 5214 |
|
sizes | MethodLength | Method length is 199 lines (max allowed is 150). | 5575 |
|
sizes | MethodLength | Method length is 214 lines (max allowed is 150). | 7252 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 8477 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 8478 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 8480 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 8481 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 8493 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 8494 |
org/apache/hadoop/hbase/regionserver/HRegionFileSystem.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 208 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 400 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 421 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 433 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 488 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 994 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1083 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1106 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1131 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1170 |
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. | 247 |
|
sizes | MethodLength | Method length is 228 lines (max allowed is 150). | 787 |
|
blocks | EmptyBlock | Must have at least one statement. | 929 |
|
sizes | MethodLength | Method length is 179 lines (max allowed is 150). | 1803 |
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/HStore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
indentation | Indentation | 'synchronized' has incorrect indentation level 6, expected level should be 8. | 1263 |
|
indentation | Indentation | 'synchronized' child has incorrect indentation level 8, expected level should be 10. | 1264 |
|
indentation | Indentation | 'synchronized rcurly' has incorrect indentation level 6, expected level should be 8. | 1265 |
|
indentation | Indentation | 'block rcurly' has incorrect indentation level 4, expected level should be 6. | 1266 |
org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | FinalClass | Class FlushHandler should be declared as final. | 306 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 501 |
org/apache/hadoop/hbase/regionserver/MetricsRegionServerSource.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 100 characters (found 113). | 303 |
org/apache/hadoop/hbase/regionserver/MetricsRegionServerWrapperImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 265 lines (max allowed is 150). | 790 |
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. | 129 |
org/apache/hadoop/hbase/regionserver/RSRpcServices.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 539 |
|
sizes | MethodLength | Method length is 161 lines (max allowed is 150). | 704 |
|
sizes | LineLength | Line is longer than 100 characters (found 102). | 1215 |
|
sizes | MethodLength | Method length is 231 lines (max allowed is 150). | 2623 |
|
sizes | MethodLength | Method length is 226 lines (max allowed is 150). | 3254 |
|
sizes | MethodLength | Method length is 250 lines (max allowed is 150). | 3488 |
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. | 90 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 155 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 203 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 214 |
org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1350 |
org/apache/hadoop/hbase/regionserver/RegionScannerImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 217 lines (max allowed is 150). | 413 |
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. | 223 |
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. | 147 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 152 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 177 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 238 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 241 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 273 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 275 |
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/StoreEngine.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
indentation | Indentation | 'block rcurly' has incorrect indentation level 4, expected level should be 6. | 407 |
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 |
org/apache/hadoop/hbase/regionserver/StoreFileReader.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 508 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 525 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 639 |
org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 211 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 239 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 294 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 440 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 494 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 502 |
org/apache/hadoop/hbase/regionserver/StoreFileWriter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | FinalClass | Class StoreFileWriter should be declared as final. | 79 |
org/apache/hadoop/hbase/regionserver/StoreScanner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 250 lines (max allowed is 150). | 540 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 774 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 907 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 934 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1186 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1221 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 1222 |
org/apache/hadoop/hbase/regionserver/StorefileRefresherChore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 90 |
org/apache/hadoop/hbase/regionserver/StripeMultiFileWriter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 198 |
org/apache/hadoop/hbase/regionserver/StripeStoreFileManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
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 |
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/TestCellFlatSet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 67 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 69 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 292 |
org/apache/hadoop/hbase/regionserver/TestCellSkipListSet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 164 |
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/TestCompaction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 609 |
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. | 314 |
org/apache/hadoop/hbase/regionserver/TestDefaultMemStore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 311 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 513 |
|
blocks | NeedBraces | 'while' construct must use '{}'s. | 1083 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 1100 |
org/apache/hadoop/hbase/regionserver/TestFSErrorsExposed.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 224 |
org/apache/hadoop/hbase/regionserver/TestFailedAppendAndSync.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 299 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 307 |
org/apache/hadoop/hbase/regionserver/TestHRegion.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1467 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 1468 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1484 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 1485 |
|
blocks | NeedBraces | 'while' construct must use '{}'s. | 4461 |
|
coding | EmptyStatement | Empty statement. | 4462 |
|
design | FinalClass | Class PutThread should be declared as final. | 4650 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 4822 |
|
blocks | EmptyBlock | Must have at least one statement. | 4885 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 5035 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 5046 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 5047 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 5065 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 5740 |
org/apache/hadoop/hbase/regionserver/TestHRegionFileSystem.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 287 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 293 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 299 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 332 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 344 |
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. | 184 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 247 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 253 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 273 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 274 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 379 |
org/apache/hadoop/hbase/regionserver/TestHStoreFile.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 153 lines (max allowed is 150). | 424 |
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. | 475 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 494 |
org/apache/hadoop/hbase/regionserver/TestMultiColumnScanner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 123 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 160 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 180 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 188 |
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/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/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. | 201 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 206 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 214 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 228 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 339 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 345 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 432 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 605 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 621 |
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. | 634 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 636 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 653 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 655 |
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. | 760 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 787 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1032 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1055 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1056 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1097 |
|
blocks | EmptyBlock | Must have at least one statement. | 1143 |
org/apache/hadoop/hbase/regionserver/TestTags.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 304 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 615 |
org/apache/hadoop/hbase/regionserver/TestWALLockup.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'while' construct must use '{}'s. | 148 |
org/apache/hadoop/hbase/regionserver/TestWALMonotonicallyIncreasingSeqId.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 235 |
org/apache/hadoop/hbase/regionserver/TestWalAndCompactingMemStoreFlush.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 222 lines (max allowed is 150). | 137 |
|
sizes | MethodLength | Method length is 260 lines (max allowed is 150). | 363 |
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. | 136 |
|
design | VisibilityModifier | Variable 'earliestPutTs' must be private and have accessor methods. | 138 |
|
design | VisibilityModifier | Variable 'latestPutTs' must be private and have accessor methods. | 140 |
|
design | VisibilityModifier | Variable 'maxSeqId' must be private and have accessor methods. | 142 |
|
design | VisibilityModifier | Variable 'maxMVCCReadpoint' must be private and have accessor methods. | 144 |
|
design | VisibilityModifier | Variable 'maxTagsLength' must be private and have accessor methods. | 146 |
|
design | VisibilityModifier | Variable 'minSeqIdToKeep' must be private and have accessor methods. | 148 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 215 |
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/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 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 206 |
org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 201 |
org/apache/hadoop/hbase/regionserver/http/RSStatusServlet.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 49 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 50 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 51 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 52 |
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/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. | 243 |
org/apache/hadoop/hbase/regionserver/wal/AbstractTestLogRolling.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 290 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 291 |
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). | 237 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 586 |
org/apache/hadoop/hbase/regionserver/wal/TestLogRolling.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 178 lines (max allowed is 150). | 109 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 572 |
org/apache/hadoop/hbase/regionserver/wal/TestLogRollingNoCluster.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 182 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 183 |
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/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/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. | 200 |
org/apache/hadoop/hbase/replication/regionserver/TestReplicationSink.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 126 |
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 145 |
|
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 154 |
org/apache/hadoop/hbase/rest/RESTServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | VisibilityModifier | Variable 'conf' must be private and have accessor methods. | 111 |
|
blocks | EmptyBlock | Must have at least one statement. | 243 |
|
sizes | MethodLength | Method length is 159 lines (max allowed is 150). | 255 |
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. | 290 |
org/apache/hadoop/hbase/rest/RowSpec.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 70 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 87 |
org/apache/hadoop/hbase/rest/client/Client.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 328 |
org/apache/hadoop/hbase/rest/model/ScannerModel.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | VisibilityModifier | Variable 'type' must be private and have accessor methods. | 133 |
|
design | VisibilityModifier | Variable 'value' must be private and have accessor methods. | 135 |
|
design | VisibilityModifier | Variable 'op' must be private and have accessor methods. | 137 |
|
design | VisibilityModifier | Variable 'type' must be private and have accessor methods. | 208 |
|
design | VisibilityModifier | Variable 'op' must be private and have accessor methods. | 210 |
|
design | VisibilityModifier | Variable 'value' must be private and have accessor methods. | 214 |
|
design | VisibilityModifier | Variable 'filters' must be private and have accessor methods. | 216 |
|
design | VisibilityModifier | Variable 'limit' must be private and have accessor methods. | 218 |
|
design | VisibilityModifier | Variable 'offset' must be private and have accessor methods. | 220 |
|
design | VisibilityModifier | Variable 'family' must be private and have accessor methods. | 222 |
|
design | VisibilityModifier | Variable 'qualifier' must be private and have accessor methods. | 224 |
|
design | VisibilityModifier | Variable 'ifMissing' must be private and have accessor methods. | 226 |
|
design | VisibilityModifier | Variable 'latestVersion' must be private and have accessor methods. | 228 |
|
design | VisibilityModifier | Variable 'minColumn' must be private and have accessor methods. | 230 |
|
design | VisibilityModifier | Variable 'minColumnInclusive' must be private and have accessor methods. | 232 |
|
design | VisibilityModifier | Variable 'maxColumn' must be private and have accessor methods. | 234 |
|
design | VisibilityModifier | Variable 'maxColumnInclusive' must be private and have accessor methods. | 236 |
|
design | VisibilityModifier | Variable 'dropDependentColumn' must be private and have accessor methods. | 238 |
|
design | VisibilityModifier | Variable 'chance' must be private and have accessor methods. | 240 |
|
design | VisibilityModifier | Variable 'prefixes' must be private and have accessor methods. | 242 |
|
design | VisibilityModifier | Variable 'timestamps' must be private and have accessor methods. | 246 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 841 |
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). | 57 |
|
sizes | LineLength | Line is longer than 100 characters (found 112). | 58 |
org/apache/hadoop/hbase/rest/model/VersionModel.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 75 |
org/apache/hadoop/hbase/security/AbstractTestSecureIPC.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 344 |
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. | 216 |
org/apache/hadoop/hbase/security/SaslUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 34 |
org/apache/hadoop/hbase/security/SecurityUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 26 |
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. | 165 |
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 length is 169 lines (max allowed is 150). | 435 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 549 |
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 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 262 |
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 length is 243 lines (max allowed is 150). | 1344 |
|
sizes | MethodLength | Method 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 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 length is 162 lines (max allowed is 150). | 235 |
|
sizes | MethodLength | Method 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/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 |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 151 lines (max allowed is 150). | 137 |
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. | 364 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 400 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 627 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 672 |
org/apache/hadoop/hbase/security/visibility/ExpAsStringVisibilityLabelServiceImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 430 |
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 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 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. | 121 |
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 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 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. | 578 |
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. | 349 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 414 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 511 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 523 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 587 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1438 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1440 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1516 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1522 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1652 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1790 |
|
sizes | LineLength | Line is longer than 100 characters (found 106). | 2018 |
|
sizes | LineLength | Line is longer than 100 characters (found 110). | 2019 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2096 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2248 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2257 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2284 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2293 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 3103 |
org/apache/hadoop/hbase/shaded/protobuf/ResponseConverter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 255 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 265 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 295 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 305 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 414 |
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 |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 290 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 291 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 552 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 555 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 559 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 562 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 828 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 829 |
|
sizes | MethodLength | Method length is 191 lines (max allowed is 150). | 965 |
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 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 132 |
org/apache/hadoop/hbase/snapshot/RestoreSnapshotHelper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 369 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 409 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 424 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 439 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 601 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 776 |
org/apache/hadoop/hbase/snapshot/SnapshotDescriptionUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 444 |
org/apache/hadoop/hbase/snapshot/SnapshotInfo.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
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. | 339 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 485 |
org/apache/hadoop/hbase/snapshot/SnapshotManifest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 346 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 472 |
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'. | 636 |
|
misc | UpperEll | Should use uppercase 'L'. | 695 |
|
misc | UpperEll | Should use uppercase 'L'. | 699 |
|
misc | UpperEll | Should use uppercase 'L'. | 702 |
|
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 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1626 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1633 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 1634 |
org/apache/hadoop/hbase/test/IntegrationTestBigLinkedListWithVisibility.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 336 |
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/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). | 109 |
org/apache/hadoop/hbase/tool/MapreduceTestingShim.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 104 |
org/apache/hadoop/hbase/tool/TestBulkLoadHFilesSplitRecovery.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
coding | InnerAssignment | Inner assignments should be avoided. | 246 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 549 |
org/apache/hadoop/hbase/trace/OpenTelemetryClassRule.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
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/util/AbstractHBaseTool.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 282 |
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. | 1465 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1466 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1467 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1468 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1473 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1505 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1518 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1519 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1520 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1712 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1790 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1791 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1795 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 1796 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 1823 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1897 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1899 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 1901 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1927 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1929 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 1931 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1957 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1984 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2009 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 2026 |
|
blocks | NeedBraces | 'while' construct must use '{}'s. | 2037 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2050 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2063 |
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. | 52 |
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. | 59 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 59 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 60 |
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. | 456 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 566 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 646 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 900 |
|
sizes | LineLength | Line is longer than 100 characters (found 115). | 1413 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1466 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1478 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1688 |
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. | 674 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 819 |
|
design | VisibilityModifier | Variable 'regionName' must be private and have accessor methods. | 845 |
|
design | VisibilityModifier | Variable 'metaFirstKey' must be private and have accessor methods. | 846 |
|
design | VisibilityModifier | Variable 'metaLastKey' must be private and have accessor methods. | 847 |
|
design | VisibilityModifier | Variable 'storesFirstKey' must be private and have accessor methods. | 848 |
|
design | VisibilityModifier | Variable 'storesLastKey' must be private and have accessor methods. | 849 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 909 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 911 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 986 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1112 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1117 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1166 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1170 |
|
sizes | LineLength | Line is longer than 100 characters (found 118). | 1176 |
|
sizes | LineLength | Line is longer than 100 characters (found 104). | 1186 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1204 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 1359 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2039 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2081 |
|
sizes | MethodLength | Method length is 231 lines (max allowed is 150). | 2102 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2105 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 2177 |
|
design | VisibilityModifier | Variable 'errorCount' must be private and have accessor methods. | 2809 |
|
sizes | LineLength | Line is longer than 100 characters (found 101). | 3473 |
|
sizes | LineLength | Line is longer than 100 characters (found 114). | 3478 |
|
sizes | LineLength | Line is longer than 100 characters (found 103). | 3493 |
|
sizes | LineLength | Line is longer than 100 characters (found 120). | 3501 |
|
sizes | LineLength | Line is longer than 100 characters (found 103). | 3505 |
|
sizes | MethodLength | Method length is 224 lines (max allowed is 150). | 3557 |
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 |
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. | 60 |
org/apache/hadoop/hbase/util/LoadTestTool.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 189 lines (max allowed is 150). | 574 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 647 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 886 |
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. | 96 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 120 |
org/apache/hadoop/hbase/util/MultiThreadedAction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 100 characters (found 229). | 81 |
|
sizes | MethodLength | Method length is 170 lines (max allowed is 150). | 314 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 533 |
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. | 35 |
org/apache/hadoop/hbase/util/RegionMover.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | FinalClass | Class RegionMover should be declared as final. | 84 |
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. | 134 |
|
sizes | MethodLength | Method length is 194 lines (max allowed is 150). | 398 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 514 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 675 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 683 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 751 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 752 |
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 |
|---|---|---|---|---|
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 65 |
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/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. | 139 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 172 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 205 |
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/TestRegionMoverWithRSGroupEnable.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 96 |
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/ZKDataMigrator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | FinalClass | Class ZKDataMigrator should be declared as final. | 42 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 104 |
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 |
|---|---|---|---|---|
|
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. | 234 |
|
blocks | NeedBraces | 'while' construct must use '{}'s. | 849 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 1068 |
org/apache/hadoop/hbase/wal/WALPerformanceEvaluation.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 176 lines (max allowed is 150). | 178 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 319 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 322 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 343 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 395 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 525 |
org/apache/hadoop/hbase/wal/WALSplitter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 154 lines (max allowed is 150). | 280 |
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 |