The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | EmptyBlock | 1 | ![]() |
coding | HiddenField | 4 | ![]() |
MagicNumber | 2 | ![]() | |
design | DesignForExtension | 20 | ![]() |
VisibilityModifier | 10 | ![]() | |
javadoc | JavadocMethod | 10 | ![]() |
JavadocPackage | 1 | ![]() | |
JavadocStyle | 1 | ![]() | |
JavadocType | 2 | ![]() | |
JavadocVariable | 23 | ![]() | |
misc | FinalParameters | 12 | ![]() |
TodoComment | 2 | ![]() | |
modifier | ModifierOrder | 1 | ![]() |
sizes | LineLength | 18 | ![]() |
whitespace | ParenPad | 4 | ![]() |
WhitespaceAfter | 1 | ![]() | |
WhitespaceAround | 1 | ![]() |
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 |