The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | LeftCurly | 3 | ![]() |
coding | HiddenField | 28 | ![]() |
MagicNumber | 2 | ![]() | |
SimplifyBooleanExpression | 3 | ![]() | |
design | DesignForExtension | 48 | ![]() |
HideUtilityClassConstructor | 1 | ![]() | |
VisibilityModifier | 13 | ![]() | |
javadoc | JavadocMethod | 33 | ![]() |
JavadocPackage | 2 | ![]() | |
JavadocStyle | 35 | ![]() | |
JavadocType | 5 | ![]() | |
JavadocVariable | 57 | ![]() | |
misc | FinalParameters | 53 | ![]() |
TodoComment | 4 | ![]() | |
modifier | RedundantModifier | 4 | ![]() |
regexp | RegexpSingleline
|
36 | ![]() |
sizes | LineLength | 63 | ![]() |
whitespace | ParenPad | 24 | ![]() |
WhitespaceAround | 2 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 22 |
![]() |
blocks | LeftCurly | '{' at column 1 should be on the previous line. | 35 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 37 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 39 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 92). | 43 |
![]() |
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 | JavadocStyle | First sentence should end with a period. | 58 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 60 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 65 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 67 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 72 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 74 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 79 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 80 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 82 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 84 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 94 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 96 |
![]() |
misc | FinalParameters | Parameter scheme should be final. | 102 |
![]() |
coding | HiddenField | 'scheme' hides a field. | 102 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' looks like designed for extension (can be subclassed), but the method 'withScheme' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsConfiguration' final or making the method 'withScheme' static/final/abstract/empty, or adding allowed annotation for the method. | 106 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 106 |
![]() |
misc | FinalParameters | Parameter scheme should be final. | 106 |
![]() |
coding | HiddenField | 'scheme' hides a field. | 106 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 111 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 113 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 123 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 125 |
![]() |
misc | FinalParameters | Parameter host should be final. | 131 |
![]() |
coding | HiddenField | 'host' hides a field. | 131 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' looks like designed for extension (can be subclassed), but the method 'withHost' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsConfiguration' final or making the method 'withHost' static/final/abstract/empty, or adding allowed annotation for the method. | 135 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 135 |
![]() |
misc | FinalParameters | Parameter host should be final. | 135 |
![]() |
coding | HiddenField | 'host' hides a field. | 135 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 140 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 142 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 152 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 154 |
![]() |
misc | FinalParameters | Parameter port should be final. | 160 |
![]() |
coding | HiddenField | 'port' hides a field. | 160 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' looks like designed for extension (can be subclassed), but the method 'withPort' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsConfiguration' final or making the method 'withPort' static/final/abstract/empty, or adding allowed annotation for the method. | 164 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 164 |
![]() |
misc | FinalParameters | Parameter port should be final. | 164 |
![]() |
coding | HiddenField | 'port' hides a field. | 164 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 169 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 171 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 181 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 183 |
![]() |
misc | FinalParameters | Parameter path should be final. | 189 |
![]() |
coding | HiddenField | 'path' hides a field. | 189 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' looks like designed for extension (can be subclassed), but the method 'withPath' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsConfiguration' final or making the method 'withPath' static/final/abstract/empty, or adding allowed annotation for the method. | 193 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 193 |
![]() |
misc | FinalParameters | Parameter path should be final. | 193 |
![]() |
coding | HiddenField | 'path' hides a field. | 193 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 198 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 200 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 210 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 212 |
![]() |
misc | FinalParameters | Parameter user should be final. | 218 |
![]() |
coding | HiddenField | 'user' hides a field. | 218 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' looks like designed for extension (can be subclassed), but the method 'withUser' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsConfiguration' final or making the method 'withUser' static/final/abstract/empty, or adding allowed annotation for the method. | 222 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 222 |
![]() |
misc | FinalParameters | Parameter user should be final. | 222 |
![]() |
coding | HiddenField | 'user' hides a field. | 222 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 227 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 229 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 239 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 241 |
![]() |
misc | FinalParameters | Parameter password should be final. | 247 |
![]() |
coding | HiddenField | 'password' hides a field. | 247 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' looks like designed for extension (can be subclassed), but the method 'withPassword' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsConfiguration' final or making the method 'withPassword' static/final/abstract/empty, or adding allowed annotation for the method. | 251 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 251 |
![]() |
misc | FinalParameters | Parameter password should be final. | 251 |
![]() |
coding | HiddenField | 'password' hides a field. | 251 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' 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 'HdfsConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 256 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' 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 'HdfsConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 261 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 261 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' 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 'HdfsConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 267 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 267 |
![]() |
misc | FinalParameters | Parameter name should be final. | 269 |
![]() |
misc | FinalParameters | Parameter value should be final. | 269 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' 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 'HdfsConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 273 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 273 |
![]() |
misc | FinalParameters | Parameter name should be final. | 273 |
![]() |
misc | FinalParameters | Parameter value should be final. | 273 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' 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 'HdfsConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 278 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 163). | 280 |
![]() |
design | DesignForExtension | Class 'HdfsConfiguration' 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 'HdfsConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 283 |
![]() |
misc | FinalParameters | Parameter other should be final. | 284 |
![]() |
coding | SimplifyBooleanExpression | Expression can be simplified. | 288 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 251). | 292 |
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 295 |
![]() |
modifier | RedundantModifier | Redundant 'static' modifier. | 296 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 298 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 299 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 300 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 301 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 302 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 303 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 304 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 305 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 121). | 306 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 306 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 314 |
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 314 |
![]() |
misc | FinalParameters | Parameter value should be final. | 314 |
![]() |
coding | HiddenField | 'value' hides a field. | 314 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 324 |
![]() |
misc | FinalParameters | Parameter value should be final. | 325 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 24 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 26 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 27 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 28 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 29 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 19 |
![]() |
blocks | LeftCurly | '{' at column 1 should be on the previous line. | 27 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 29 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 31 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 36 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 37 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 39 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 41 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 51 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 53 |
![]() |
misc | FinalParameters | Parameter readerPath should be final. | 59 |
![]() |
coding | HiddenField | 'readerPath' hides a field. | 59 |
![]() |
design | DesignForExtension | Class 'HdfsReaderConfiguration' looks like designed for extension (can be subclassed), but the method 'withReaderPath' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsReaderConfiguration' final or making the method 'withReaderPath' static/final/abstract/empty, or adding allowed annotation for the method. | 63 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 63 |
![]() |
misc | FinalParameters | Parameter readerPath should be final. | 63 |
![]() |
coding | HiddenField | 'readerPath' hides a field. | 63 |
![]() |
design | DesignForExtension | Class 'HdfsReaderConfiguration' 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 'HdfsReaderConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 68 |
![]() |
design | DesignForExtension | Class 'HdfsReaderConfiguration' 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 'HdfsReaderConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 73 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 73 |
![]() |
design | DesignForExtension | Class 'HdfsReaderConfiguration' 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 'HdfsReaderConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 79 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 79 |
![]() |
misc | FinalParameters | Parameter name should be final. | 81 |
![]() |
misc | FinalParameters | Parameter value should be final. | 81 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 86). | 85 |
![]() |
design | DesignForExtension | Class 'HdfsReaderConfiguration' 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 'HdfsReaderConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 85 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 85 |
![]() |
misc | FinalParameters | Parameter name should be final. | 85 |
![]() |
misc | FinalParameters | Parameter value should be final. | 85 |
![]() |
design | DesignForExtension | Class 'HdfsReaderConfiguration' 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 'HdfsReaderConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 90 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 98). | 92 |
![]() |
design | DesignForExtension | Class 'HdfsReaderConfiguration' 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 'HdfsReaderConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 95 |
![]() |
misc | FinalParameters | Parameter other should be final. | 96 |
![]() |
coding | SimplifyBooleanExpression | Expression can be simplified. | 100 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 136). | 104 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 21 |
![]() |
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. | 35 |
![]() |
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 |
![]() |
coding | MagicNumber | '1000L' is a magic number. | 54 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 55 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 57 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 116). | 61 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 62 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 63 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 65 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 66 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 67 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 77 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 78 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 79 |
![]() |
misc | FinalParameters | Parameter writerPath should be final. | 85 |
![]() |
coding | HiddenField | 'writerPath' hides a field. | 85 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' looks like designed for extension (can be subclassed), but the method 'withWriterPath' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsWriterConfiguration' final or making the method 'withWriterPath' static/final/abstract/empty, or adding allowed annotation for the method. | 89 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 89 |
![]() |
misc | FinalParameters | Parameter writerPath should be final. | 89 |
![]() |
coding | HiddenField | 'writerPath' hides a field. | 89 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 94 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 96 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 106 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 108 |
![]() |
misc | FinalParameters | Parameter writerFilePrefix should be final. | 114 |
![]() |
coding | HiddenField | 'writerFilePrefix' hides a field. | 114 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 82). | 118 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' looks like designed for extension (can be subclassed), but the method 'withWriterFilePrefix' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsWriterConfiguration' final or making the method 'withWriterFilePrefix' static/final/abstract/empty, or adding allowed annotation for the method. | 118 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 118 |
![]() |
misc | FinalParameters | Parameter writerFilePrefix should be final. | 118 |
![]() |
coding | HiddenField | 'writerFilePrefix' hides a field. | 118 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 123 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 125 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 135 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 137 |
![]() |
misc | FinalParameters | Parameter linesPerFile should be final. | 143 |
![]() |
coding | HiddenField | 'linesPerFile' hides a field. | 143 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' looks like designed for extension (can be subclassed), but the method 'withLinesPerFile' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsWriterConfiguration' final or making the method 'withLinesPerFile' static/final/abstract/empty, or adding allowed annotation for the method. | 147 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 147 |
![]() |
misc | FinalParameters | Parameter linesPerFile should be final. | 147 |
![]() |
coding | HiddenField | 'linesPerFile' hides a field. | 147 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 152 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 154 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 164 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 166 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 172 |
![]() |
misc | FinalParameters | Parameter compression should be final. | 172 |
![]() |
coding | HiddenField | 'compression' hides a field. | 172 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 101). | 176 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' looks like designed for extension (can be subclassed), but the method 'withCompression' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'HdfsWriterConfiguration' final or making the method 'withCompression' static/final/abstract/empty, or adding allowed annotation for the method. | 176 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 176 |
![]() |
misc | FinalParameters | Parameter compression should be final. | 176 |
![]() |
coding | HiddenField | 'compression' hides a field. | 176 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' 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 'HdfsWriterConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 181 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' 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 'HdfsWriterConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 186 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 186 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' 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 'HdfsWriterConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 192 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 192 |
![]() |
misc | FinalParameters | Parameter name should be final. | 194 |
![]() |
misc | FinalParameters | Parameter value should be final. | 194 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 86). | 198 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' 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 'HdfsWriterConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 198 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 198 |
![]() |
misc | FinalParameters | Parameter name should be final. | 198 |
![]() |
misc | FinalParameters | Parameter value should be final. | 198 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' 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 'HdfsWriterConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 203 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 164). | 205 |
![]() |
design | DesignForExtension | Class 'HdfsWriterConfiguration' 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 'HdfsWriterConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 208 |
![]() |
misc | FinalParameters | Parameter other should be final. | 209 |
![]() |
coding | SimplifyBooleanExpression | Expression can be simplified. | 213 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 259). | 217 |
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 220 |
![]() |
modifier | RedundantModifier | Redundant 'static' modifier. | 221 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 223 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 224 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 225 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 143). | 226 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 226 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 234 |
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 234 |
![]() |
misc | FinalParameters | Parameter value should be final. | 234 |
![]() |
coding | HiddenField | 'value' hides a field. | 234 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 244 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 83). | 245 |
![]() |
misc | FinalParameters | Parameter value should be final. | 245 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 80 characters (found 89). | 65 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 91). | 69 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 69 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 71 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 73 |
![]() |
design | VisibilityModifier | Variable 'client' must be private and have accessor methods. | 73 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 74 |
![]() |
design | VisibilityModifier | Variable 'path' must be private and have accessor methods. | 74 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 75 |
![]() |
design | VisibilityModifier | Variable 'status' must be private and have accessor methods. | 75 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 77 |
![]() |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 77 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 79 |
![]() |
design | VisibilityModifier | Variable 'mapper' must be private and have accessor methods. | 79 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 80 |
![]() |
design | VisibilityModifier | Variable 'lineReaderUtil' must be private and have accessor methods. | 80 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 82 |
![]() |
design | VisibilityModifier | Variable 'hdfsConfiguration' must be private and have accessor methods. | 82 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 83 |
![]() |
design | VisibilityModifier | Variable 'streamsConfiguration' must be private and have accessor methods. | 83 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 85 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 87 |
![]() |
design | VisibilityModifier | Variable 'countersTotal' must be private and have accessor methods. | 87 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 88 |
![]() |
design | VisibilityModifier | Variable 'countersCurrent' must be private and have accessor methods. | 88 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 89 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 89). | 92 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 140). | 95 |
![]() |
misc | FinalParameters | Parameter hdfsConfiguration should be final. | 102 |
![]() |
coding | HiddenField | 'hdfsConfiguration' hides a field. | 102 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 146 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 147 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 105). | 150 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 84). | 151 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 91). | 155 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 106). | 163 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 92). | 164 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 106). | 168 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 89). | 169 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 105). | 183 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' 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 'WebHdfsPersistReader' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 188 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' 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 'WebHdfsPersistReader' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 193 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 194 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 94). | 198 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 202 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 207 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 131). | 220 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' 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 'WebHdfsPersistReader' final or making the method 'readAll' static/final/abstract/empty, or adding allowed annotation for the method. | 230 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' 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 'WebHdfsPersistReader' final or making the method 'startStream' static/final/abstract/empty, or adding allowed annotation for the method. | 243 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' 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 'WebHdfsPersistReader' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 249 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 254 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 254 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' 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 'WebHdfsPersistReader' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 266 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 266 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 266 |
![]() |
misc | FinalParameters | Parameter entry should be final. | 266 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 266 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 269 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 269 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 274 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 274 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' 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 'WebHdfsPersistReader' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 277 |
![]() |
misc | FinalParameters | Parameter sequence should be final. | 278 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' 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 'WebHdfsPersistReader' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 282 |
![]() |
misc | FinalParameters | Parameter start should be final. | 283 |
![]() |
misc | FinalParameters | Parameter end should be final. | 283 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' 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 'WebHdfsPersistReader' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 287 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReader' looks like designed for extension (can be subclassed), but the method 'getDatumStatusCounter' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'WebHdfsPersistReader' final or making the method 'getDatumStatusCounter' static/final/abstract/empty, or adding allowed annotation for the method. | 292 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 36 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 95). | 42 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 42 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 46 |
![]() |
misc | FinalParameters | Parameter reader should be final. | 46 |
![]() |
coding | HiddenField | 'reader' hides a field. | 46 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistReaderTask' 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 'WebHdfsPersistReaderTask' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 50 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 55 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 55 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 59 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 59 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 84). | 63 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 63 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 99). | 64 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 65 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 171). | 68 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 71 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 105). | 74 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 77 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 88 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 88 |
![]() |
coding | MagicNumber | '15' is a magic number. | 113 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 80 characters (found 111). | 57 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 59 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 91). | 61 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 61 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 63 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 64 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 65 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 66 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 68 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 69 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 71 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 72 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 73 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 74 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 76 |
![]() |
design | VisibilityModifier | Variable 'terminate' must be private and have accessor methods. | 76 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 78 |
![]() |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 78 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 80 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 81 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 83 |
![]() |
design | VisibilityModifier | Variable 'hdfsConfiguration' must be private and have accessor methods. | 83 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 85 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 140). | 86 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 89 |
![]() |
misc | FinalParameters | Parameter hdfsConfiguration should be final. | 89 |
![]() |
coding | HiddenField | 'hdfsConfiguration' hides a field. | 89 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 99 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 89). | 105 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 116 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 125 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 134 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 105). | 137 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 84). | 138 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 93). | 143 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 108). | 149 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 94). | 150 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 108). | 154 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 91). | 155 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 109). | 171 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistWriter' 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 'WebHdfsPersistWriter' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 176 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistWriter' 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 'WebHdfsPersistWriter' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 181 |
![]() |
misc | FinalParameters | Parameter streamsDatum should be final. | 182 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 185 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 186 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 191 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 135). | 204 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 211 |
![]() |
misc | FinalParameters | Parameter line should be final. | 211 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 100). | 220 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 101). | 226 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistWriter' 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 'WebHdfsPersistWriter' final or making the method 'flush' static/final/abstract/empty, or adding allowed annotation for the method. | 234 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 242 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 113). | 249 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 95). | 251 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 251 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 113). | 267 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 97). | 272 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 272 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 99). | 273 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistWriter' 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 'WebHdfsPersistWriter' final or making the method 'close' static/final/abstract/empty, or adding allowed annotation for the method. | 289 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistWriter' 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 'WebHdfsPersistWriter' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 299 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 300 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 91). | 304 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistWriter' 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 'WebHdfsPersistWriter' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 307 |
![]() |
design | DesignForExtension | Class 'WebHdfsPersistWriter' looks like designed for extension (can be subclassed), but the method 'getDatumStatusCounter' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'WebHdfsPersistWriter' final or making the method 'getDatumStatusCounter' static/final/abstract/empty, or adding allowed annotation for the method. | 321 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 28 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 95). | 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 'WebHdfsPersistWriterTask' 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 'WebHdfsPersistWriterTask' 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 |
![]() |
whitespace | WhitespaceAround | '{' is not followed by whitespace. | 56 |
![]() |
whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 56 |