The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | EmptyBlock | 1 | ![]() |
LeftCurly | 1 | ![]() | |
coding | HiddenField | 20 | ![]() |
MagicNumber | 2 | ![]() | |
SimplifyBooleanExpression | 1 | ![]() | |
design | DesignForExtension | 34 | ![]() |
VisibilityModifier | 10 | ![]() | |
javadoc | JavadocMethod | 21 | ![]() |
JavadocPackage | 2 | ![]() | |
JavadocStyle | 25 | ![]() | |
JavadocType | 3 | ![]() | |
JavadocVariable | 24 | ![]() | |
misc | FinalParameters | 33 | ![]() |
TodoComment | 2 | ![]() | |
modifier | ModifierOrder | 1 | ![]() |
regexp | RegexpSingleline
|
24 | ![]() |
sizes | LineLength | 22 | ![]() |
whitespace | ParenPad | 4 | ![]() |
WhitespaceAfter | 1 | ![]() | |
WhitespaceAround | 1 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 19 |
![]() |
blocks | LeftCurly | '{' at column 1 should be on the previous line. | 32 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 34 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 36 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 41 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 43 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 48 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 50 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 55 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 57 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 62 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 64 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 69 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 71 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 76 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 78 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 83 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 85 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 90 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 91 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 93 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 95 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 105 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 107 |
![]() |
misc | FinalParameters | Parameter rootdir should be final. | 113 |
![]() |
coding | HiddenField | 'rootdir' hides a field. | 113 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'withRootdir' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'withRootdir' static/final/abstract/empty, or adding allowed annotation for the method. | 117 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 117 |
![]() |
misc | FinalParameters | Parameter rootdir should be final. | 117 |
![]() |
coding | HiddenField | 'rootdir' hides a field. | 117 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 122 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 124 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 134 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 136 |
![]() |
misc | FinalParameters | Parameter parent should be final. | 142 |
![]() |
coding | HiddenField | 'parent' hides a field. | 142 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'withParent' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'withParent' static/final/abstract/empty, or adding allowed annotation for the method. | 146 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 146 |
![]() |
misc | FinalParameters | Parameter parent should be final. | 146 |
![]() |
coding | HiddenField | 'parent' hides a field. | 146 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 151 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 153 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 163 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 165 |
![]() |
misc | FinalParameters | Parameter rootserver should be final. | 171 |
![]() |
coding | HiddenField | 'rootserver' hides a field. | 171 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'withRootserver' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'withRootserver' static/final/abstract/empty, or adding allowed annotation for the method. | 175 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 175 |
![]() |
misc | FinalParameters | Parameter rootserver should be final. | 175 |
![]() |
coding | HiddenField | 'rootserver' hides a field. | 175 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 180 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 182 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 192 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 194 |
![]() |
misc | FinalParameters | Parameter quorum should be final. | 200 |
![]() |
coding | HiddenField | 'quorum' hides a field. | 200 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'withQuorum' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'withQuorum' static/final/abstract/empty, or adding allowed annotation for the method. | 204 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 204 |
![]() |
misc | FinalParameters | Parameter quorum should be final. | 204 |
![]() |
coding | HiddenField | 'quorum' hides a field. | 204 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 209 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 211 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 221 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 223 |
![]() |
misc | FinalParameters | Parameter clientPort should be final. | 229 |
![]() |
coding | HiddenField | 'clientPort' hides a field. | 229 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'withClientPort' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'withClientPort' static/final/abstract/empty, or adding allowed annotation for the method. | 233 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 233 |
![]() |
misc | FinalParameters | Parameter clientPort should be final. | 233 |
![]() |
coding | HiddenField | 'clientPort' hides a field. | 233 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 238 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 240 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 250 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 252 |
![]() |
misc | FinalParameters | Parameter table should be final. | 258 |
![]() |
coding | HiddenField | 'table' hides a field. | 258 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'withTable' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'withTable' static/final/abstract/empty, or adding allowed annotation for the method. | 262 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 262 |
![]() |
misc | FinalParameters | Parameter table should be final. | 262 |
![]() |
coding | HiddenField | 'table' hides a field. | 262 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 267 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 269 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 279 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 281 |
![]() |
misc | FinalParameters | Parameter family should be final. | 287 |
![]() |
coding | HiddenField | 'family' hides a field. | 287 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'withFamily' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'withFamily' static/final/abstract/empty, or adding allowed annotation for the method. | 291 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 291 |
![]() |
misc | FinalParameters | Parameter family should be final. | 291 |
![]() |
coding | HiddenField | 'family' hides a field. | 291 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 296 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 298 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 308 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 310 |
![]() |
misc | FinalParameters | Parameter qualifier should be final. | 316 |
![]() |
coding | HiddenField | 'qualifier' hides a field. | 316 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'withQualifier' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'withQualifier' static/final/abstract/empty, or adding allowed annotation for the method. | 320 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 320 |
![]() |
misc | FinalParameters | Parameter qualifier should be final. | 320 |
![]() |
coding | HiddenField | 'qualifier' hides a field. | 320 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'toString' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 325 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'getAdditionalProperties' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 330 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 330 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'setAdditionalProperty' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 336 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 336 |
![]() |
misc | FinalParameters | Parameter name should be final. | 338 |
![]() |
misc | FinalParameters | Parameter value should be final. | 338 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 342 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'withAdditionalProperty' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 342 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 342 |
![]() |
misc | FinalParameters | Parameter name should be final. | 342 |
![]() |
misc | FinalParameters | Parameter value should be final. | 342 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'hashCode' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 347 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 210). | 349 |
![]() |
design | DesignForExtension | Class 'HbaseConfiguration' looks like designed for extension (can be subclassed), but the method 'equals' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbaseConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 352 |
![]() |
misc | FinalParameters | Parameter other should be final. | 353 |
![]() |
coding | SimplifyBooleanExpression | Expression can be simplified. | 357 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 339). | 361 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 42 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 44 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 91). | 46 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 46 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 48 |
![]() |
design | VisibilityModifier | Variable 'lock' must be private and have accessor methods. | 48 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 50 |
![]() |
design | VisibilityModifier | Variable 'running' must be private and have accessor methods. | 50 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 52 |
![]() |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 52 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 54 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 56 |
![]() |
design | VisibilityModifier | Variable 'config' must be private and have accessor methods. | 56 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 58 |
![]() |
design | VisibilityModifier | Variable 'props' must be private and have accessor methods. | 58 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 60 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 62 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 83). | 64 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 69 |
![]() |
misc | FinalParameters | Parameter config should be final. | 69 |
![]() |
coding | HiddenField | 'config' hides a field. | 69 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'setConfig' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'setConfig' static/final/abstract/empty, or adding allowed annotation for the method. | 73 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 73 |
![]() |
misc | FinalParameters | Parameter config should be final. | 73 |
![]() |
coding | HiddenField | 'config' hides a field. | 73 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'getId' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 77 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'startStream' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'startStream' static/final/abstract/empty, or adding allowed annotation for the method. | 82 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'readAll' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'readAll' static/final/abstract/empty, or adding allowed annotation for the method. | 93 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'readCurrent' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 98 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'constructQueue' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'constructQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 116 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 116 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'readNew' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 120 |
![]() |
misc | FinalParameters | Parameter bigInteger should be final. | 121 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'readRange' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 125 |
![]() |
misc | FinalParameters | Parameter dateTime should be final. | 126 |
![]() |
misc | FinalParameters | Parameter dateTime2 should be final. | 126 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'isRunning' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 130 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'prepare' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 135 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 136 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 100). | 142 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 102). | 143 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'cleanUp' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 151 |
![]() |
design | DesignForExtension | Class 'HbasePersistReader' looks like designed for extension (can be subclassed), but the method 'write' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReader' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 162 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 162 |
![]() |
misc | FinalParameters | Parameter streamsDatum should be final. | 162 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 24 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 95). | 26 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 26 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 28 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 30 |
![]() |
misc | FinalParameters | Parameter reader should be final. | 30 |
![]() |
coding | HiddenField | 'reader' hides a field. | 30 |
![]() |
design | DesignForExtension | Class 'HbasePersistReaderTask' looks like designed for extension (can be subclassed), but the method 'run' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistReaderTask' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 34 |
![]() |
whitespace | WhitespaceAround | 'while' is not followed by whitespace. | 37 |
![]() |
blocks | EmptyBlock | Must have at least one statement. | 37 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 80 characters (found 87). | 50 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 52 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 89). | 54 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 54 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 56 |
![]() |
design | VisibilityModifier | Variable 'connection' must be private and have accessor methods. | 56 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 57 |
![]() |
design | VisibilityModifier | Variable 'pool' must be private and have accessor methods. | 57 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 58 |
![]() |
design | VisibilityModifier | Variable 'table' must be private and have accessor methods. | 58 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 59 |
![]() |
design | VisibilityModifier | Variable 'descriptor' must be private and have accessor methods. | 59 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 61 |
![]() |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 61 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 63 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 65 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 82). | 68 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 72 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 80 |
![]() |
misc | FinalParameters | Parameter hbaseConfiguration should be final. | 81 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 86 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 88 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 100). | 95 |
![]() |
coding | MagicNumber | '1000' is a magic number. | 96 |
![]() |
coding | MagicNumber | '1000' is a magic number. | 98 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 103). | 129 |
![]() |
design | DesignForExtension | Class 'HbasePersistWriter' looks like designed for extension (can be subclassed), but the method 'getId' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistWriter' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 134 |
![]() |
design | DesignForExtension | Class 'HbasePersistWriter' looks like designed for extension (can be subclassed), but the method 'write' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistWriter' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 139 |
![]() |
misc | FinalParameters | Parameter streamsDatum should be final. | 140 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 150 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 150 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 86). | 152 |
![]() |
whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 152 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 88). | 160 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 88). | 163 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 88). | 177 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 88). | 180 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 189 |
![]() |
design | DesignForExtension | Class 'HbasePersistWriter' looks like designed for extension (can be subclassed), but the method 'flush' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistWriter' final or making the method 'flush' static/final/abstract/empty, or adding allowed annotation for the method. | 194 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 194 |
![]() |
design | DesignForExtension | Class 'HbasePersistWriter' looks like designed for extension (can be subclassed), but the method 'close' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistWriter' final or making the method 'close' static/final/abstract/empty, or adding allowed annotation for the method. | 198 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 198 |
![]() |
design | DesignForExtension | Class 'HbasePersistWriter' looks like designed for extension (can be subclassed), but the method 'prepare' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistWriter' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 202 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 203 |
![]() |
design | DesignForExtension | Class 'HbasePersistWriter' looks like designed for extension (can be subclassed), but the method 'cleanUp' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistWriter' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 218 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 28 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 93). | 34 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 34 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 36 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 38 |
![]() |
misc | FinalParameters | Parameter writer should be final. | 38 |
![]() |
coding | HiddenField | 'writer' hides a field. | 38 |
![]() |
design | DesignForExtension | Class 'HbasePersistWriterTask' looks like designed for extension (can be subclassed), but the method 'run' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HbasePersistWriterTask' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 42 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 46 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 46 |