The following document contains the results of Checkstyle 6.11.2 with sun_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | LeftCurly | 3 | |
| NeedBraces | 1 | ||
| coding | HiddenField | 22 | |
| MagicNumber | 2 | ||
| SimplifyBooleanExpression | 3 | ||
| design | DesignForExtension | 52 | |
| VisibilityModifier | 5 | ||
| javadoc | JavadocMethod | 25 | |
| JavadocPackage | 2 | ||
| JavadocStyle | 13 | ||
| JavadocType | 4 | ||
| JavadocVariable | 32 | ||
| misc | FinalParameters | 42 | |
| modifier | RedundantModifier | 2 | |
| regexp | RegexpSingleline
|
16 | |
| sizes | LineLength | 40 | |
| whitespace | ParenPad | 8 | |
| WhitespaceAfter | 1 | ||
| WhitespaceAround | 6 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| javadoc | JavadocType | Missing a Javadoc comment. | 22 | |
| blocks | LeftCurly | '{' at column 1 should be on the previous line. | 31 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 33 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 36 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 42 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 45 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 51 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 53 | |
| sizes | LineLength | Line is longer than 80 characters (found 102). | 57 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 58 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 60 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 65 | |
| sizes | LineLength | Line is longer than 80 characters (found 85). | 66 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 68 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 71 | |
| design | DesignForExtension | Method 'getKey' is not designed for extension - needs to be abstract, final or empty. | 75 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 81 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 84 | |
| design | DesignForExtension | Method 'setKey' is not designed for extension - needs to be abstract, final or empty. | 88 | |
| misc | FinalParameters | Parameter key should be final. | 90 | |
| coding | HiddenField | 'key' hides a field. | 90 | |
| design | DesignForExtension | Method 'withKey' is not designed for extension - needs to be abstract, final or empty. | 94 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 94 | |
| misc | FinalParameters | Parameter key should be final. | 94 | |
| coding | HiddenField | 'key' hides a field. | 94 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 99 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 102 | |
| design | DesignForExtension | Method 'getSecretKey' is not designed for extension - needs to be abstract, final or empty. | 106 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 112 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 115 | |
| design | DesignForExtension | Method 'setSecretKey' is not designed for extension - needs to be abstract, final or empty. | 119 | |
| misc | FinalParameters | Parameter secretKey should be final. | 121 | |
| coding | HiddenField | 'secretKey' hides a field. | 121 | |
| design | DesignForExtension | Method 'withSecretKey' is not designed for extension - needs to be abstract, final or empty. | 125 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 125 | |
| misc | FinalParameters | Parameter secretKey should be final. | 125 | |
| coding | HiddenField | 'secretKey' hides a field. | 125 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 130 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 132 | |
| design | DesignForExtension | Method 'getProtocol' is not designed for extension - needs to be abstract, final or empty. | 136 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 142 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 144 | |
| design | DesignForExtension | Method 'setProtocol' is not designed for extension - needs to be abstract, final or empty. | 148 | |
| misc | FinalParameters | Parameter protocol should be final. | 150 | |
| coding | HiddenField | 'protocol' hides a field. | 150 | |
| sizes | LineLength | Line is longer than 80 characters (found 86). | 154 | |
| design | DesignForExtension | Method 'withProtocol' is not designed for extension - needs to be abstract, final or empty. | 154 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 154 | |
| misc | FinalParameters | Parameter protocol should be final. | 154 | |
| coding | HiddenField | 'protocol' hides a field. | 154 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 159 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 161 | |
| design | DesignForExtension | Method 'getRegion' is not designed for extension - needs to be abstract, final or empty. | 165 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 171 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 173 | |
| design | DesignForExtension | Method 'setRegion' is not designed for extension - needs to be abstract, final or empty. | 177 | |
| misc | FinalParameters | Parameter region should be final. | 179 | |
| coding | HiddenField | 'region' hides a field. | 179 | |
| design | DesignForExtension | Method 'withRegion' is not designed for extension - needs to be abstract, final or empty. | 183 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 183 | |
| misc | FinalParameters | Parameter region should be final. | 183 | |
| coding | HiddenField | 'region' hides a field. | 183 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 188 | |
| design | DesignForExtension | Method 'getAdditionalProperties' is not designed for extension - needs to be abstract, final or empty. | 193 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 193 | |
| design | DesignForExtension | Method 'setAdditionalProperty' is not designed for extension - needs to be abstract, final or empty. | 199 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 199 | |
| misc | FinalParameters | Parameter name should be final. | 201 | |
| misc | FinalParameters | Parameter value should be final. | 201 | |
| sizes | LineLength | Line is longer than 80 characters (found 83). | 205 | |
| design | DesignForExtension | Method 'withAdditionalProperty' is not designed for extension - needs to be abstract, final or empty. | 205 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 205 | |
| misc | FinalParameters | Parameter name should be final. | 205 | |
| misc | FinalParameters | Parameter value should be final. | 205 | |
| design | DesignForExtension | Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 210 | |
| sizes | LineLength | Line is longer than 80 characters (found 141). | 212 | |
| design | DesignForExtension | Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 215 | |
| misc | FinalParameters | Parameter other should be final. | 216 | |
| coding | SimplifyBooleanExpression | Expression can be simplified. | 220 | |
| sizes | LineLength | Line is longer than 80 characters (found 213). | 224 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 227 | |
| modifier | RedundantModifier | Redundant 'static' modifier. | 228 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 230 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 231 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 232 | |
| sizes | LineLength | Line is longer than 80 characters (found 131). | 233 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 233 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 241 | |
| modifier | RedundantModifier | Redundant 'private' modifier. | 241 | |
| misc | FinalParameters | Parameter value should be final. | 241 | |
| coding | HiddenField | 'value' hides a field. | 241 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 251 | |
| misc | FinalParameters | Parameter value should be final. | 252 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 80 characters (found 81). | 55 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 57 | |
| sizes | LineLength | Line is longer than 80 characters (found 91). | 59 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 59 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 61 | |
| design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 61 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 63 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 65 | |
| sizes | LineLength | Line is longer than 80 characters (found 96). | 67 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 67 | |
| design | VisibilityModifier | Variable 'pollInterval' must be private and have accessor methods. | 67 | |
| 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 | |
| sizes | LineLength | Line is longer than 80 characters (found 95). | 76 | |
| coding | HiddenField | 'config' hides a field. | 79 | |
| sizes | LineLength | Line is longer than 80 characters (found 108). | 80 | |
| sizes | LineLength | Line is longer than 80 characters (found 81). | 85 | |
| misc | FinalParameters | Parameter config should be final. | 87 | |
| coding | HiddenField | 'config' hides a field. | 87 | |
| javadoc | JavadocMethod | Expected @param tag for 'config'. | 87 | |
| design | DesignForExtension | Method 'setConfig' is not designed for extension - needs to be abstract, final or empty. | 92 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 92 | |
| misc | FinalParameters | Parameter config should be final. | 92 | |
| coding | HiddenField | 'config' hides a field. | 92 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 96 | |
| design | DesignForExtension | Method 'startStream' is not designed for extension - needs to be abstract, final or empty. | 101 | |
| sizes | LineLength | Line is longer than 80 characters (found 91). | 110 | |
| whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 110 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 110 | |
| sizes | LineLength | Line is longer than 80 characters (found 88). | 112 | |
| whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 114 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 114 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 114 | |
| sizes | LineLength | Line is longer than 80 characters (found 90). | 115 | |
| design | DesignForExtension | Method 'readAll' is not designed for extension - needs to be abstract, final or empty. | 123 | |
| design | DesignForExtension | Method 'readCurrent' is not designed for extension - needs to be abstract, final or empty. | 128 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 128 | |
| whitespace | WhitespaceAround | 'synchronized' is not followed by whitespace. | 131 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 131 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 131 | |
| design | DesignForExtension | Method 'readNew' is not designed for extension - needs to be abstract, final or empty. | 138 | |
| misc | FinalParameters | Parameter bigInteger should be final. | 139 | |
| design | DesignForExtension | Method 'readRange' is not designed for extension - needs to be abstract, final or empty. | 143 | |
| misc | FinalParameters | Parameter dateTime should be final. | 144 | |
| misc | FinalParameters | Parameter dateTime2 should be final. | 144 | |
| design | DesignForExtension | Method 'isRunning' is not designed for extension - needs to be abstract, final or empty. | 148 | |
| design | DesignForExtension | Method 'prepare' is not designed for extension - needs to be abstract, final or empty. | 153 | |
| misc | FinalParameters | Parameter configurationObject should be final. | 154 | |
| sizes | LineLength | Line is longer than 80 characters (found 99). | 158 | |
| sizes | LineLength | Line is longer than 80 characters (found 82). | 161 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 164 | |
| sizes | LineLength | Line is longer than 80 characters (found 86). | 165 | |
| design | DesignForExtension | Method 'cleanUp' is not designed for extension - needs to be abstract, final or empty. | 171 | |
| whitespace | WhitespaceAround | 'while' is not followed by whitespace. | 174 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 174 | |
| coding | MagicNumber | '5' is a magic number. | 176 | |
| whitespace | WhitespaceAround | '{' is not followed by whitespace. | 177 | |
| whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 177 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| javadoc | JavadocStyle | First sentence should end with a period. | 39 | |
| sizes | LineLength | Line is longer than 80 characters (found 95). | 45 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 45 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 47 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 48 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 49 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 51 | |
| sizes | LineLength | Line is longer than 80 characters (found 94). | 53 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 53 | |
| sizes | LineLength | Line is longer than 80 characters (found 99). | 58 | |
| misc | FinalParameters | Parameter reader should be final. | 58 | |
| coding | HiddenField | 'reader' hides a field. | 58 | |
| javadoc | JavadocMethod | Expected @param tag for 'reader'. | 58 | |
| misc | FinalParameters | Parameter streamName should be final. | 58 | |
| coding | HiddenField | 'streamName' hides a field. | 58 | |
| javadoc | JavadocMethod | Expected @param tag for 'streamName'. | 58 | |
| misc | FinalParameters | Parameter shardId should be final. | 58 | |
| coding | HiddenField | 'shardId' hides a field. | 58 | |
| javadoc | JavadocMethod | Expected @param tag for 'shardId'. | 58 | |
| design | DesignForExtension | Method 'run' is not designed for extension - needs to be abstract, final or empty. | 64 | |
| sizes | LineLength | Line is longer than 80 characters (found 102). | 72 | |
| whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 76 | |
| sizes | LineLength | Line is longer than 80 characters (found 171). | 87 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 105 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 105 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocVariable | Missing a Javadoc comment. | 57 | |
| sizes | LineLength | Line is longer than 80 characters (found 91). | 59 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 59 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 61 | |
| design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 61 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 63 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 65 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 67 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 69 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 71 | |
| design | VisibilityModifier | Variable 'client' must be private and have accessor methods. | 71 | |
| sizes | LineLength | Line is longer than 80 characters (found 95). | 74 | |
| coding | HiddenField | 'config' hides a field. | 77 | |
| sizes | LineLength | Line is longer than 80 characters (found 108). | 78 | |
| sizes | LineLength | Line is longer than 80 characters (found 81). | 83 | |
| misc | FinalParameters | Parameter config should be final. | 85 | |
| coding | HiddenField | 'config' hides a field. | 85 | |
| javadoc | JavadocMethod | Expected @param tag for 'config'. | 85 | |
| design | DesignForExtension | Method 'setConfig' is not designed for extension - needs to be abstract, final or empty. | 90 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 90 | |
| misc | FinalParameters | Parameter config should be final. | 90 | |
| coding | HiddenField | 'config' hides a field. | 90 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 94 | |
| design | DesignForExtension | Method 'write' is not designed for extension - needs to be abstract, final or empty. | 99 | |
| misc | FinalParameters | Parameter entry should be final. | 100 | |
| sizes | LineLength | Line is longer than 80 characters (found 106). | 102 | |
| design | DesignForExtension | Method 'prepare' is not designed for extension - needs to be abstract, final or empty. | 116 | |
| misc | FinalParameters | Parameter configurationObject should be final. | 117 | |
| sizes | LineLength | Line is longer than 80 characters (found 99). | 121 | |
| sizes | LineLength | Line is longer than 80 characters (found 82). | 124 | |
| sizes | LineLength | Line is longer than 80 characters (found 86). | 128 | |
| design | DesignForExtension | Method 'cleanUp' is not designed for extension - needs to be abstract, final or empty. | 135 | |
| coding | MagicNumber | '5' is a magic number. | 138 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocType | Missing a Javadoc comment. | 22 | |
| blocks | LeftCurly | '{' at column 1 should be on the previous line. | 30 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 32 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 36 | |
| sizes | LineLength | Line is longer than 80 characters (found 85). | 37 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 40 | |
| design | DesignForExtension | Method 'getStreams' is not designed for extension - needs to be abstract, final or empty. | 44 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 51 | |
| design | DesignForExtension | Method 'setStreams' is not designed for extension - needs to be abstract, final or empty. | 55 | |
| misc | FinalParameters | Parameter streams should be final. | 57 | |
| coding | HiddenField | 'streams' hides a field. | 57 | |
| design | DesignForExtension | Method 'withStreams' is not designed for extension - needs to be abstract, final or empty. | 61 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 61 | |
| misc | FinalParameters | Parameter streams should be final. | 61 | |
| coding | HiddenField | 'streams' hides a field. | 61 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 66 | |
| design | DesignForExtension | Method 'getAdditionalProperties' is not designed for extension - needs to be abstract, final or empty. | 71 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 71 | |
| design | DesignForExtension | Method 'setAdditionalProperty' is not designed for extension - needs to be abstract, final or empty. | 77 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 77 | |
| misc | FinalParameters | Parameter name should be final. | 79 | |
| misc | FinalParameters | Parameter value should be final. | 79 | |
| sizes | LineLength | Line is longer than 80 characters (found 89). | 83 | |
| design | DesignForExtension | Method 'withAdditionalProperty' is not designed for extension - needs to be abstract, final or empty. | 83 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 83 | |
| misc | FinalParameters | Parameter name should be final. | 83 | |
| misc | FinalParameters | Parameter value should be final. | 83 | |
| design | DesignForExtension | Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 88 | |
| sizes | LineLength | Line is longer than 80 characters (found 95). | 90 | |
| design | DesignForExtension | Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 93 | |
| misc | FinalParameters | Parameter other should be final. | 94 | |
| coding | SimplifyBooleanExpression | Expression can be simplified. | 98 | |
| sizes | LineLength | Line is longer than 80 characters (found 130). | 102 |
| 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 | JavadocVariable | Missing a Javadoc comment. | 29 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 32 | |
| sizes | LineLength | Line is longer than 80 characters (found 85). | 33 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 36 | |
| design | DesignForExtension | Method 'getStream' is not designed for extension - needs to be abstract, final or empty. | 40 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 47 | |
| design | DesignForExtension | Method 'setStream' is not designed for extension - needs to be abstract, final or empty. | 51 | |
| misc | FinalParameters | Parameter stream should be final. | 53 | |
| coding | HiddenField | 'stream' hides a field. | 53 | |
| design | DesignForExtension | Method 'withStream' is not designed for extension - needs to be abstract, final or empty. | 57 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 57 | |
| misc | FinalParameters | Parameter stream should be final. | 57 | |
| coding | HiddenField | 'stream' hides a field. | 57 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 62 | |
| design | DesignForExtension | Method 'getAdditionalProperties' is not designed for extension - needs to be abstract, final or empty. | 67 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 67 | |
| design | DesignForExtension | Method 'setAdditionalProperty' is not designed for extension - needs to be abstract, final or empty. | 73 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 73 | |
| misc | FinalParameters | Parameter name should be final. | 75 | |
| misc | FinalParameters | Parameter value should be final. | 75 | |
| sizes | LineLength | Line is longer than 80 characters (found 89). | 79 | |
| design | DesignForExtension | Method 'withAdditionalProperty' is not designed for extension - needs to be abstract, final or empty. | 79 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 79 | |
| misc | FinalParameters | Parameter name should be final. | 79 | |
| misc | FinalParameters | Parameter value should be final. | 79 | |
| design | DesignForExtension | Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 84 | |
| sizes | LineLength | Line is longer than 80 characters (found 94). | 86 | |
| design | DesignForExtension | Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 89 | |
| misc | FinalParameters | Parameter other should be final. | 90 | |
| coding | SimplifyBooleanExpression | Expression can be simplified. | 94 | |
| sizes | LineLength | Line is longer than 80 characters (found 128). | 98 |