The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | LeftCurly | 1 | ![]() |
coding | HiddenField | 17 | ![]() |
MagicNumber | 3 | ![]() | |
SimplifyBooleanExpression | 1 | ![]() | |
design | DesignForExtension | 30 | ![]() |
VisibilityModifier | 3 | ![]() | |
javadoc | JavadocMethod | 17 | ![]() |
JavadocPackage | 2 | ![]() | |
JavadocStyle | 15 | ![]() | |
JavadocType | 1 | ![]() | |
JavadocVariable | 20 | ![]() | |
misc | FinalParameters | 29 | ![]() |
regexp | RegexpSingleline
|
12 | ![]() |
sizes | LineLength | 20 | ![]() |
whitespace | ParenPad | 3 | ![]() |
WhitespaceAfter | 3 | ![]() |
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. | 28 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 30 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 32 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 37 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 39 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 44 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 46 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 51 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 53 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 58 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 59 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 61 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 63 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 73 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 75 |
![]() |
misc | FinalParameters | Parameter brokerlist should be final. | 81 |
![]() |
coding | HiddenField | 'brokerlist' hides a field. | 81 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' looks like designed for extension (can be subclassed), but the method 'withBrokerlist' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KafkaConfiguration' final or making the method 'withBrokerlist' static/final/abstract/empty, or adding allowed annotation for the method. | 85 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 85 |
![]() |
misc | FinalParameters | Parameter brokerlist should be final. | 85 |
![]() |
coding | HiddenField | 'brokerlist' hides a field. | 85 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 90 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 92 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 102 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 104 |
![]() |
misc | FinalParameters | Parameter zkconnect should be final. | 110 |
![]() |
coding | HiddenField | 'zkconnect' hides a field. | 110 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' looks like designed for extension (can be subclassed), but the method 'withZkconnect' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KafkaConfiguration' final or making the method 'withZkconnect' static/final/abstract/empty, or adding allowed annotation for the method. | 114 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 114 |
![]() |
misc | FinalParameters | Parameter zkconnect should be final. | 114 |
![]() |
coding | HiddenField | 'zkconnect' hides a field. | 114 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 119 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 121 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 131 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 133 |
![]() |
misc | FinalParameters | Parameter topic should be final. | 139 |
![]() |
coding | HiddenField | 'topic' hides a field. | 139 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' looks like designed for extension (can be subclassed), but the method 'withTopic' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KafkaConfiguration' final or making the method 'withTopic' static/final/abstract/empty, or adding allowed annotation for the method. | 143 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 143 |
![]() |
misc | FinalParameters | Parameter topic should be final. | 143 |
![]() |
coding | HiddenField | 'topic' hides a field. | 143 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 148 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 150 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 160 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 162 |
![]() |
misc | FinalParameters | Parameter groupId should be final. | 168 |
![]() |
coding | HiddenField | 'groupId' hides a field. | 168 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' looks like designed for extension (can be subclassed), but the method 'withGroupId' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KafkaConfiguration' final or making the method 'withGroupId' static/final/abstract/empty, or adding allowed annotation for the method. | 172 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 172 |
![]() |
misc | FinalParameters | Parameter groupId should be final. | 172 |
![]() |
coding | HiddenField | 'groupId' hides a field. | 172 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' 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 'KafkaConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 177 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' 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 'KafkaConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 182 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 182 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' 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 'KafkaConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 188 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 188 |
![]() |
misc | FinalParameters | Parameter name should be final. | 190 |
![]() |
misc | FinalParameters | Parameter value should be final. | 190 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 194 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' 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 'KafkaConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 194 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 194 |
![]() |
misc | FinalParameters | Parameter name should be final. | 194 |
![]() |
misc | FinalParameters | Parameter value should be final. | 194 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' 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 'KafkaConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 199 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 146). | 201 |
![]() |
design | DesignForExtension | Class 'KafkaConfiguration' 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 'KafkaConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 204 |
![]() |
misc | FinalParameters | Parameter other should be final. | 205 |
![]() |
coding | SimplifyBooleanExpression | Expression can be simplified. | 209 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 223). | 213 |
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 |