The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | LeftCurly | 3 |  Error |
| NeedBraces | 1 | Error | |
| coding | HiddenField | 22 | Error |
| MagicNumber | 2 | Error | |
| SimplifyBooleanExpression | 3 | Error | |
| design | DesignForExtension | 40 | Error |
| VisibilityModifier | 5 | Error | |
| javadoc | JavadocMethod | 25 | Error |
| JavadocPackage | 2 | Error | |
| JavadocStyle | 13 | Error | |
| JavadocType | 4 | Error | |
| JavadocVariable | 32 | Error | |
| misc | FinalParameters | 42 | Error |
| modifier | RedundantModifier | 2 | Error |
| regexp | RegexpSingleline
|
16 | Error |
| sizes | LineLength | 40 | Error |
| whitespace | ParenPad | 8 | Error |
| WhitespaceAfter | 1 | Error | |
| WhitespaceAround | 6 | Error |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 21 |
| Error |
blocks | LeftCurly | '{' at column 1 should be on the previous line. | 30 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 32 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 35 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 40 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 43 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 48 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 50 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 102). | 53 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 54 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 56 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 60 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 61 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 63 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 66 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 75 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 78 |
| Error |
misc | FinalParameters | Parameter key should be final. | 83 |
| Error |
coding | HiddenField | 'key' hides a field. | 83 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' looks like designed for extension (can be subclassed), but the method 'withKey' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KinesisConfiguration' final or making the method 'withKey' static/final/abstract/empty, or adding allowed annotation for the method. | 87 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 87 |
| Error |
misc | FinalParameters | Parameter key should be final. | 87 |
| Error |
coding | HiddenField | 'key' hides a field. | 87 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 92 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 95 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 104 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 107 |
| Error |
misc | FinalParameters | Parameter secretKey should be final. | 112 |
| Error |
coding | HiddenField | 'secretKey' hides a field. | 112 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' looks like designed for extension (can be subclassed), but the method 'withSecretKey' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KinesisConfiguration' final or making the method 'withSecretKey' static/final/abstract/empty, or adding allowed annotation for the method. | 116 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 116 |
| Error |
misc | FinalParameters | Parameter secretKey should be final. | 116 |
| Error |
coding | HiddenField | 'secretKey' hides a field. | 116 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 121 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 123 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 132 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 134 |
| Error |
misc | FinalParameters | Parameter protocol should be final. | 139 |
| Error |
coding | HiddenField | 'protocol' hides a field. | 139 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 143 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' looks like designed for extension (can be subclassed), but the method 'withProtocol' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KinesisConfiguration' final or making the method 'withProtocol' static/final/abstract/empty, or adding allowed annotation for the method. | 143 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 143 |
| Error |
misc | FinalParameters | Parameter protocol should be final. | 143 |
| Error |
coding | HiddenField | 'protocol' hides a field. | 143 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 148 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 150 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 159 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 161 |
| Error |
misc | FinalParameters | Parameter region should be final. | 166 |
| Error |
coding | HiddenField | 'region' hides a field. | 166 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' looks like designed for extension (can be subclassed), but the method 'withRegion' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KinesisConfiguration' final or making the method 'withRegion' static/final/abstract/empty, or adding allowed annotation for the method. | 170 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 170 |
| Error |
misc | FinalParameters | Parameter region should be final. | 170 |
| Error |
coding | HiddenField | 'region' hides a field. | 170 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' 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 'KinesisConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 175 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' 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 'KinesisConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 180 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 180 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' 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 'KinesisConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 185 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 185 |
| Error |
misc | FinalParameters | Parameter name should be final. | 186 |
| Error |
misc | FinalParameters | Parameter value should be final. | 186 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 83). | 190 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' 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 'KinesisConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 190 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 190 |
| Error |
misc | FinalParameters | Parameter name should be final. | 190 |
| Error |
misc | FinalParameters | Parameter value should be final. | 190 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' 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 'KinesisConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 195 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 141). | 197 |
| Error |
design | DesignForExtension | Class 'KinesisConfiguration' 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 'KinesisConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 200 |
| Error |
misc | FinalParameters | Parameter other should be final. | 201 |
| Error |
coding | SimplifyBooleanExpression | Expression can be simplified. | 205 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 213). | 209 |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 212 |
| Error |
modifier | RedundantModifier | Redundant 'static' modifier. | 213 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 215 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 216 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 217 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 131). | 218 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 218 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 226 |
| Error |
modifier | RedundantModifier | Redundant 'private' modifier. | 226 |
| Error |
misc | FinalParameters | Parameter value should be final. | 226 |
| Error |
coding | HiddenField | 'value' hides a field. | 226 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 236 |
| Error |
misc | FinalParameters | Parameter value should be final. | 237 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 55 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 57 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 91). | 59 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 59 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 61 |
| Error |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 61 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 63 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 65 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 96). | 67 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
| Error |
design | VisibilityModifier | Variable 'pollInterval' must be private and have accessor methods. | 67 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 69 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 71 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 73 |
| Error |
design | VisibilityModifier | Variable 'client' must be private and have accessor methods. | 73 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 76 |
| Error |
coding | HiddenField | 'config' hides a field. | 79 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 108). | 80 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 85 |
| Error |
misc | FinalParameters | Parameter config should be final. | 87 |
| Error |
coding | HiddenField | 'config' hides a field. | 87 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'config'. | 87 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' looks like designed for extension (can be subclassed), but the method 'setConfig' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KinesisPersistReader' final or making the method 'setConfig' static/final/abstract/empty, or adding allowed annotation for the method. | 92 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 92 |
| Error |
misc | FinalParameters | Parameter config should be final. | 92 |
| Error |
coding | HiddenField | 'config' hides a field. | 92 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' 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 'KinesisPersistReader' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 96 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' 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 'KinesisPersistReader' final or making the method 'startStream' static/final/abstract/empty, or adding allowed annotation for the method. | 101 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 91). | 110 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 110 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 110 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 88). | 112 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 114 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 114 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 114 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 90). | 115 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' 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 'KinesisPersistReader' final or making the method 'readAll' static/final/abstract/empty, or adding allowed annotation for the method. | 123 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' 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 'KinesisPersistReader' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 128 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 128 |
| Error |
whitespace | WhitespaceAround | 'synchronized' is not followed by whitespace. | 131 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 131 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 131 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' 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 'KinesisPersistReader' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 138 |
| Error |
misc | FinalParameters | Parameter bigInteger should be final. | 139 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' 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 'KinesisPersistReader' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 143 |
| Error |
misc | FinalParameters | Parameter dateTime should be final. | 144 |
| Error |
misc | FinalParameters | Parameter dateTime2 should be final. | 144 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' 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 'KinesisPersistReader' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 148 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' 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 'KinesisPersistReader' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 153 |
| Error |
misc | FinalParameters | Parameter configurationObject should be final. | 154 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 99). | 158 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 82). | 161 |
| Error |
blocks | NeedBraces | 'if' construct must use '{}'s. | 164 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 165 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReader' 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 'KinesisPersistReader' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 171 |
| Error |
whitespace | WhitespaceAround | 'while' is not followed by whitespace. | 174 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 174 |
| Error |
coding | MagicNumber | '5' is a magic number. | 176 |
| Error |
whitespace | WhitespaceAround | '{' is not followed by whitespace. | 177 |
| Error |
whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 177 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 39 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 45 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 45 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 47 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 48 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 49 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 51 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 94). | 53 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 53 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 99). | 58 |
| Error |
misc | FinalParameters | Parameter reader should be final. | 58 |
| Error |
coding | HiddenField | 'reader' hides a field. | 58 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'reader'. | 58 |
| Error |
misc | FinalParameters | Parameter streamName should be final. | 58 |
| Error |
coding | HiddenField | 'streamName' hides a field. | 58 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'streamName'. | 58 |
| Error |
misc | FinalParameters | Parameter shardId should be final. | 58 |
| Error |
coding | HiddenField | 'shardId' hides a field. | 58 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'shardId'. | 58 |
| Error |
design | DesignForExtension | Class 'KinesisPersistReaderTask' 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 'KinesisPersistReaderTask' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 64 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 102). | 72 |
| Error |
whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 76 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 171). | 87 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 105 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 105 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 57 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 91). | 59 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 59 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 61 |
| Error |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 61 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 63 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 65 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 69 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 71 |
| Error |
design | VisibilityModifier | Variable 'client' must be private and have accessor methods. | 71 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 74 |
| Error |
coding | HiddenField | 'config' hides a field. | 77 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 108). | 78 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 83 |
| Error |
misc | FinalParameters | Parameter config should be final. | 85 |
| Error |
coding | HiddenField | 'config' hides a field. | 85 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'config'. | 85 |
| Error |
design | DesignForExtension | Class 'KinesisPersistWriter' looks like designed for extension (can be subclassed), but the method 'setConfig' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KinesisPersistWriter' final or making the method 'setConfig' static/final/abstract/empty, or adding allowed annotation for the method. | 90 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 90 |
| Error |
misc | FinalParameters | Parameter config should be final. | 90 |
| Error |
coding | HiddenField | 'config' hides a field. | 90 |
| Error |
design | DesignForExtension | Class 'KinesisPersistWriter' 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 'KinesisPersistWriter' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 94 |
| Error |
design | DesignForExtension | Class 'KinesisPersistWriter' 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 'KinesisPersistWriter' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 99 |
| Error |
misc | FinalParameters | Parameter entry should be final. | 100 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 106). | 102 |
| Error |
design | DesignForExtension | Class 'KinesisPersistWriter' 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 'KinesisPersistWriter' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 116 |
| Error |
misc | FinalParameters | Parameter configurationObject should be final. | 117 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 99). | 121 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 82). | 124 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 128 |
| Error |
design | DesignForExtension | Class 'KinesisPersistWriter' 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 'KinesisPersistWriter' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 135 |
| Error |
coding | MagicNumber | '5' is a magic number. | 138 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 21 |
| Error |
blocks | LeftCurly | '{' at column 1 should be on the previous line. | 29 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 31 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 34 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 35 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 38 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 48 |
| Error |
misc | FinalParameters | Parameter streams should be final. | 53 |
| Error |
coding | HiddenField | 'streams' hides a field. | 53 |
| Error |
design | DesignForExtension | Class 'KinesisReaderConfiguration' looks like designed for extension (can be subclassed), but the method 'withStreams' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KinesisReaderConfiguration' final or making the method 'withStreams' static/final/abstract/empty, or adding allowed annotation for the method. | 57 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 57 |
| Error |
misc | FinalParameters | Parameter streams should be final. | 57 |
| Error |
coding | HiddenField | 'streams' hides a field. | 57 |
| Error |
design | DesignForExtension | Class 'KinesisReaderConfiguration' 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 'KinesisReaderConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 62 |
| Error |
design | DesignForExtension | Class 'KinesisReaderConfiguration' 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 'KinesisReaderConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 67 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 67 |
| Error |
design | DesignForExtension | Class 'KinesisReaderConfiguration' 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 'KinesisReaderConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 72 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 72 |
| Error |
misc | FinalParameters | Parameter name should be final. | 73 |
| Error |
misc | FinalParameters | Parameter value should be final. | 73 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 89). | 77 |
| Error |
design | DesignForExtension | Class 'KinesisReaderConfiguration' 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 'KinesisReaderConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 77 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 77 |
| Error |
misc | FinalParameters | Parameter name should be final. | 77 |
| Error |
misc | FinalParameters | Parameter value should be final. | 77 |
| Error |
design | DesignForExtension | Class 'KinesisReaderConfiguration' 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 'KinesisReaderConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 82 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 84 |
| Error |
design | DesignForExtension | Class 'KinesisReaderConfiguration' 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 'KinesisReaderConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 87 |
| Error |
misc | FinalParameters | Parameter other should be final. | 88 |
| Error |
coding | SimplifyBooleanExpression | Expression can be simplified. | 92 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 130). | 96 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 18 |
| Error |
blocks | LeftCurly | '{' at column 1 should be on the previous line. | 26 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 28 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 30 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 31 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 34 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 44 |
| Error |
misc | FinalParameters | Parameter stream should be final. | 49 |
| Error |
coding | HiddenField | 'stream' hides a field. | 49 |
| Error |
design | DesignForExtension | Class 'KinesisWriterConfiguration' looks like designed for extension (can be subclassed), but the method 'withStream' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'KinesisWriterConfiguration' final or making the method 'withStream' static/final/abstract/empty, or adding allowed annotation for the method. | 53 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 53 |
| Error |
misc | FinalParameters | Parameter stream should be final. | 53 |
| Error |
coding | HiddenField | 'stream' hides a field. | 53 |
| Error |
design | DesignForExtension | Class 'KinesisWriterConfiguration' 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 'KinesisWriterConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 58 |
| Error |
design | DesignForExtension | Class 'KinesisWriterConfiguration' 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 'KinesisWriterConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 63 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 63 |
| Error |
design | DesignForExtension | Class 'KinesisWriterConfiguration' 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 'KinesisWriterConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 68 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 68 |
| Error |
misc | FinalParameters | Parameter name should be final. | 69 |
| Error |
misc | FinalParameters | Parameter value should be final. | 69 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 89). | 73 |
| Error |
design | DesignForExtension | Class 'KinesisWriterConfiguration' 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 'KinesisWriterConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 73 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 73 |
| Error |
misc | FinalParameters | Parameter name should be final. | 73 |
| Error |
misc | FinalParameters | Parameter value should be final. | 73 |
| Error |
design | DesignForExtension | Class 'KinesisWriterConfiguration' 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 'KinesisWriterConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 78 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 94). | 80 |
| Error |
design | DesignForExtension | Class 'KinesisWriterConfiguration' 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 'KinesisWriterConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 83 |
| Error |
misc | FinalParameters | Parameter other should be final. | 84 |
| Error |
coding | SimplifyBooleanExpression | Expression can be simplified. | 88 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 128). | 92 |
Info
Warnings