The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
coding | HiddenField | 9 | ![]() |
MagicNumber | 3 | ![]() | |
design | DesignForExtension | 20 | ![]() |
VisibilityModifier | 3 | ![]() | |
javadoc | JavadocMethod | 10 | ![]() |
JavadocPackage | 1 | ![]() | |
JavadocStyle | 3 | ![]() | |
JavadocVariable | 19 | ![]() | |
misc | FinalParameters | 16 | ![]() |
sizes | LineLength | 16 | ![]() |
whitespace | ParenPad | 3 | ![]() |
WhitespaceAfter | 3 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 54 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 89). | 56 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 56 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 58 |
![]() |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 58 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 60 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 62 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 64 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 66 |
![]() |
design | VisibilityModifier | Variable 'inStreams' must be private and have accessor methods. | 66 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 68 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 83). | 71 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 75 |
![]() |
misc | FinalParameters | Parameter persistQueue should be final. | 82 |
![]() |
coding | HiddenField | 'persistQueue' hides a field. | 82 |
![]() |
javadoc | JavadocMethod | Expected @param tag for 'persistQueue'. | 82 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 84 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReader' 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 'KafkaPersistReader' final or making the method 'setConfig' static/final/abstract/empty, or adding allowed annotation for the method. | 88 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 88 |
![]() |
misc | FinalParameters | Parameter config should be final. | 88 |
![]() |
coding | HiddenField | 'config' hides a field. | 88 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReader' 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 'KafkaPersistReader' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 92 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReader' 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 'KafkaPersistReader' final or making the method 'startStream' static/final/abstract/empty, or adding allowed annotation for the method. | 97 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 128). | 110 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReader' 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 'KafkaPersistReader' final or making the method 'readAll' static/final/abstract/empty, or adding allowed annotation for the method. | 118 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReader' 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 'KafkaPersistReader' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 123 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReader' 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 'KafkaPersistReader' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 128 |
![]() |
misc | FinalParameters | Parameter bigInteger should be final. | 129 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReader' 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 'KafkaPersistReader' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 133 |
![]() |
misc | FinalParameters | Parameter dateTime should be final. | 134 |
![]() |
misc | FinalParameters | Parameter dateTime2 should be final. | 134 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReader' 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 'KafkaPersistReader' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 138 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 88). | 143 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 143 |
![]() |
misc | FinalParameters | Parameter zookeeper should be final. | 143 |
![]() |
misc | FinalParameters | Parameter groupId should be final. | 143 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 154 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReader' 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 'KafkaPersistReader' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 158 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 161 |
![]() |
coding | MagicNumber | '5' is a magic number. | 163 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 30 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 93). | 36 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 36 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 38 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 39 |
![]() |
whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 39 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 95). | 41 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 41 |
![]() |
misc | FinalParameters | Parameter reader should be final. | 41 |
![]() |
coding | HiddenField | 'reader' hides a field. | 41 |
![]() |
misc | FinalParameters | Parameter stream should be final. | 41 |
![]() |
whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 41 |
![]() |
coding | HiddenField | 'stream' hides a field. | 41 |
![]() |
design | DesignForExtension | Class 'KafkaPersistReaderTask' 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 'KafkaPersistReaderTask' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 46 |
![]() |
whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 49 |
![]() |
coding | MagicNumber | '100' is a magic number. | 57 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 80 characters (found 89). | 43 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 45 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 89). | 47 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 47 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 49 |
![]() |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 49 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 51 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 53 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 55 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 83). | 58 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 62 |
![]() |
misc | FinalParameters | Parameter persistQueue should be final. | 69 |
![]() |
coding | HiddenField | 'persistQueue' hides a field. | 69 |
![]() |
javadoc | JavadocMethod | Expected @param tag for 'persistQueue'. | 69 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 71 |
![]() |
design | DesignForExtension | Class 'KafkaPersistWriter' 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 'KafkaPersistWriter' final or making the method 'setConfig' static/final/abstract/empty, or adding allowed annotation for the method. | 75 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 75 |
![]() |
misc | FinalParameters | Parameter config should be final. | 75 |
![]() |
coding | HiddenField | 'config' hides a field. | 75 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 79 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 82). | 87 |
![]() |
coding | HiddenField | 'config' hides a field. | 90 |
![]() |
design | DesignForExtension | Class 'KafkaPersistWriter' looks like designed for extension (can be subclassed), but the method 'stop' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KafkaPersistWriter' final or making the method 'stop' static/final/abstract/empty, or adding allowed annotation for the method. | 97 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 97 |
![]() |
design | DesignForExtension | Class 'KafkaPersistWriter' looks like designed for extension (can be subclassed), but the method 'setPersistQueue' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KafkaPersistWriter' final or making the method 'setPersistQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 101 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 101 |
![]() |
misc | FinalParameters | Parameter persistQueue should be final. | 101 |
![]() |
coding | HiddenField | 'persistQueue' hides a field. | 101 |
![]() |
design | DesignForExtension | Class 'KafkaPersistWriter' looks like designed for extension (can be subclassed), but the method 'getPersistQueue' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KafkaPersistWriter' final or making the method 'getPersistQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 105 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 105 |
![]() |
design | DesignForExtension | Class 'KafkaPersistWriter' 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 'KafkaPersistWriter' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 109 |
![]() |
design | DesignForExtension | Class 'KafkaPersistWriter' 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 'KafkaPersistWriter' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 114 |
![]() |
misc | FinalParameters | Parameter entry should be final. | 115 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 92). | 123 |
![]() |
design | DesignForExtension | Class 'KafkaPersistWriter' 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 'KafkaPersistWriter' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 132 |
![]() |
design | DesignForExtension | Class 'KafkaPersistWriter' 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 'KafkaPersistWriter' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 137 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 138 |
![]() |
design | DesignForExtension | Class 'KafkaPersistWriter' 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 'KafkaPersistWriter' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 142 |
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 'KafkaPersistWriterTask' 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 'KafkaPersistWriterTask' 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 |
![]() |
coding | MagicNumber | '100' is a magic number. | 55 |