The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
coding | HiddenField | 7 | Error |
design | DesignForExtension | 14 | Error |
VisibilityModifier | 3 | Error | |
javadoc | JavadocMethod | 19 | Error |
JavadocPackage | 1 | Error | |
JavadocStyle | 1 | Error | |
JavadocType | 1 | Error | |
JavadocVariable | 17 | Error | |
misc | FinalParameters | 45 | Error |
NewlineAtEndOfFile | 1 | Error | |
sizes | LineLength | 36 | Error |
whitespace | EmptyForIteratorPad | 2 | Error |
ParenPad | 31 | Error |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocPackage | Missing package-info.java file. | |
Error | sizes | LineLength | Line is longer than 80 characters (found 102). | 39 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'getSourceDirectory' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'getSourceDirectory' static/final/abstract/empty, or adding allowed annotation for the method. | 41 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 41 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'getSourcePaths' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'getSourcePaths' static/final/abstract/empty, or adding allowed annotation for the method. | 45 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 45 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 49 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 50 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 51 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 52 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'getExclusions' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'getExclusions' static/final/abstract/empty, or adding allowed annotation for the method. | 54 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 54 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'setExclusions' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'setExclusions' static/final/abstract/empty, or adding allowed annotation for the method. | 58 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 58 |
Error | misc | FinalParameters | Parameter exclusions should be final. | 58 |
Error | coding | HiddenField | 'exclusions' hides a field. | 58 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 62 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'getMaxDepth' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'getMaxDepth' static/final/abstract/empty, or adding allowed annotation for the method. | 64 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 64 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'setSourceDirectory' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'setSourceDirectory' static/final/abstract/empty, or adding allowed annotation for the method. | 68 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 68 |
Error | misc | FinalParameters | Parameter sourceDirectory should be final. | 68 |
Error | coding | HiddenField | 'sourceDirectory' hides a field. | 68 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'setSourcePaths' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'setSourcePaths' static/final/abstract/empty, or adding allowed annotation for the method. | 72 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 72 |
Error | misc | FinalParameters | Parameter sourcePaths should be final. | 72 |
Error | coding | HiddenField | 'sourcePaths' hides a field. | 72 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'setTargetDirectory' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'setTargetDirectory' static/final/abstract/empty, or adding allowed annotation for the method. | 76 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 76 |
Error | misc | FinalParameters | Parameter targetDirectory should be final. | 76 |
Error | coding | HiddenField | 'targetDirectory' hides a field. | 76 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'getTargetDirectory' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'getTargetDirectory' static/final/abstract/empty, or adding allowed annotation for the method. | 80 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'getSource' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'getSource' static/final/abstract/empty, or adding allowed annotation for the method. | 85 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 88 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 91 |
Error | design | DesignForExtension | Class 'StreamsHiveGenerationConfig' looks like designed for extension (can be subclassed), but the method 'setMaxDepth' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveGenerationConfig' final or making the method 'setMaxDepth' static/final/abstract/empty, or adding allowed annotation for the method. | 99 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 99 |
Error | misc | FinalParameters | Parameter maxDepth should be final. | 99 |
Error | coding | HiddenField | 'maxDepth' hides a field. | 99 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | sizes | LineLength | Line is longer than 80 characters (found 118). | 51 |
Error | sizes | LineLength | Line is longer than 80 characters (found 99). | 57 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 57 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 59 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 61 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 63 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 65 |
Error | javadoc | JavadocStyle | First sentence should end with a period. | 67 |
Error | sizes | LineLength | Line is longer than 80 characters (found 136). | 71 |
Error | misc | FinalParameters | Parameter args should be final. | 75 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 81 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 81 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 84 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 84 |
Error | sizes | LineLength | Line is longer than 80 characters (found 105). | 91 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 95 |
Error | misc | FinalParameters | Parameter config should be final. | 95 |
Error | coding | HiddenField | 'config' hides a field. | 95 |
Error | design | DesignForExtension | Class 'StreamsHiveResourceGenerator' 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 'StreamsHiveResourceGenerator' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 99 |
Error | misc | FinalParameters | Parameter config should be final. | 112 |
Error | coding | HiddenField | 'config' hides a field. | 112 |
Error | sizes | LineLength | Line is longer than 80 characters (found 105). | 133 |
Error | whitespace | EmptyForIteratorPad | ';' is followed by whitespace. | 133 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 136 |
Error | sizes | LineLength | Line is longer than 80 characters (found 91). | 138 |
Error | sizes | LineLength | Line is longer than 80 characters (found 107). | 142 |
Error | misc | FinalParameters | Parameter schema should be final. | 163 |
Error | misc | FinalParameters | Parameter resourceId should be final. | 163 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 170 |
Error | sizes | LineLength | Line is longer than 80 characters (found 84). | 173 |
Error | sizes | LineLength | Line is longer than 80 characters (found 89). | 175 |
Error | sizes | LineLength | Line is longer than 80 characters (found 122). | 184 |
Error | design | DesignForExtension | Class 'StreamsHiveResourceGenerator' looks like designed for extension (can be subclassed), but the method 'appendRootObject' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveResourceGenerator' final or making the method 'appendRootObject' static/final/abstract/empty, or adding allowed annotation for the method. | 184 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 184 |
Error | misc | FinalParameters | Parameter builder should be final. | 184 |
Error | misc | FinalParameters | Parameter schema should be final. | 184 |
Error | misc | FinalParameters | Parameter resourceId should be final. | 184 |
Error | misc | FinalParameters | Parameter seperator should be final. | 184 |
Error | sizes | LineLength | Line is longer than 80 characters (found 88). | 185 |
Error | sizes | LineLength | Line is longer than 80 characters (found 92). | 186 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 186 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 187 |
Error | sizes | LineLength | Line is longer than 80 characters (found 138). | 192 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 192 |
Error | misc | FinalParameters | Parameter builder should be final. | 192 |
Error | misc | FinalParameters | Parameter schema should be final. | 192 |
Error | misc | FinalParameters | Parameter fieldId should be final. | 192 |
Error | misc | FinalParameters | Parameter fieldType should be final. | 192 |
Error | misc | FinalParameters | Parameter seperator should be final. | 192 |
Error | sizes | LineLength | Line is longer than 80 characters (found 141). | 201 |
Error | design | DesignForExtension | Class 'StreamsHiveResourceGenerator' looks like designed for extension (can be subclassed), but the method 'appendArrayItems' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsHiveResourceGenerator' final or making the method 'appendArrayItems' static/final/abstract/empty, or adding allowed annotation for the method. | 201 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 201 |
Error | misc | FinalParameters | Parameter builder should be final. | 201 |
Error | misc | FinalParameters | Parameter schema should be final. | 201 |
Error | misc | FinalParameters | Parameter fieldId should be final. | 201 |
Error | misc | FinalParameters | Parameter itemsNode should be final. | 201 |
Error | misc | FinalParameters | Parameter seperator should be final. | 201 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 204 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 204 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 207 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 210 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 210 |
Error | sizes | LineLength | Line is longer than 80 characters (found 88). | 212 |
Error | sizes | LineLength | Line is longer than 80 characters (found 91). | 216 |
Error | sizes | LineLength | Line is longer than 80 characters (found 86). | 219 |
Error | sizes | LineLength | Line is longer than 80 characters (found 138). | 229 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 229 |
Error | misc | FinalParameters | Parameter builder should be final. | 229 |
Error | misc | FinalParameters | Parameter schema should be final. | 229 |
Error | misc | FinalParameters | Parameter fieldId should be final. | 229 |
Error | misc | FinalParameters | Parameter fieldType should be final. | 229 |
Error | misc | FinalParameters | Parameter seperator should be final. | 229 |
Error | sizes | LineLength | Line is longer than 80 characters (found 140). | 240 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 240 |
Error | misc | FinalParameters | Parameter builder should be final. | 240 |
Error | misc | FinalParameters | Parameter schema should be final. | 240 |
Error | misc | FinalParameters | Parameter fieldId should be final. | 240 |
Error | misc | FinalParameters | Parameter fieldNode should be final. | 240 |
Error | misc | FinalParameters | Parameter seperator should be final. | 240 |
Error | sizes | LineLength | Line is longer than 80 characters (found 90). | 252 |
Error | sizes | LineLength | Line is longer than 80 characters (found 145). | 259 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 259 |
Error | misc | FinalParameters | Parameter builder should be final. | 259 |
Error | misc | FinalParameters | Parameter schema should be final. | 259 |
Error | misc | FinalParameters | Parameter fieldId should be final. | 259 |
Error | misc | FinalParameters | Parameter propertiesNode should be final. | 259 |
Error | misc | FinalParameters | Parameter seperator should be final. | 259 |
Error | sizes | LineLength | Line is longer than 80 characters (found 93). | 264 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 264 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 264 |
Error | sizes | LineLength | Line is longer than 80 characters (found 132). | 289 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 289 |
Error | misc | FinalParameters | Parameter builder should be final. | 289 |
Error | misc | FinalParameters | Parameter schema should be final. | 289 |
Error | misc | FinalParameters | Parameter propertiesNode should be final. | 289 |
Error | misc | FinalParameters | Parameter seperator should be final. | 289 |
Error | whitespace | EmptyForIteratorPad | ';' is followed by whitespace. | 294 |
Error | sizes | LineLength | Line is longer than 80 characters (found 86). | 297 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 297 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 300 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 304 |
Error | sizes | LineLength | Line is longer than 80 characters (found 127). | 305 |
Error | sizes | LineLength | Line is longer than 80 characters (found 101). | 312 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 313 |
Error | sizes | LineLength | Line is longer than 80 characters (found 135). | 314 |
Error | sizes | LineLength | Line is longer than 80 characters (found 125). | 321 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 334 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 334 |
Error | misc | FinalParameters | Parameter fieldId should be final. | 334 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 334 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 338 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 338 |
Error | misc | FinalParameters | Parameter fieldType should be final. | 338 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 338 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 339 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 339 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | misc | NewlineAtEndOfFile | File does not end with a newline. | |
Error | javadoc | JavadocType | Missing a Javadoc comment. | 37 |
Error | sizes | LineLength | Line is longer than 80 characters (found 103). | 47 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 47 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 49 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 51 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 54 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 54 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 54 |
Error | sizes | LineLength | Line is longer than 80 characters (found 85). | 57 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 57 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 57 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 57 |
Error | design | VisibilityModifier | Variable 'sourceDirectory' must be private and have accessor methods. | 58 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 60 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 60 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 60 |
Error | design | VisibilityModifier | Variable 'sourcePaths' must be private and have accessor methods. | 61 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 63 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 63 |
Error | design | VisibilityModifier | Variable 'targetDirectory' must be private and have accessor methods. | 64 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 77 |
Error | sizes | LineLength | Line is longer than 80 characters (found 105). | 84 |