The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
File | ![]() |
![]() |
![]() |
---|---|---|---|
org/apache/streams/cassandra/CassandraClient.java | 0 | 0 | 42 |
org/apache/streams/cassandra/CassandraPersistReader.java | 0 | 0 | 72 |
org/apache/streams/cassandra/CassandraPersistWriter.java | 0 | 0 | 69 |
Category | Rule | Violations | Severity |
---|---|---|---|
coding | HiddenField | 7 | ![]() |
MagicNumber | 6 | ![]() | |
SimplifyBooleanExpression | 1 | ![]() | |
design | DesignForExtension | 29 | ![]() |
VisibilityModifier | 8 | ![]() | |
imports | UnusedImports
|
13 | ![]() |
javadoc | JavadocMethod | 23 | ![]() |
JavadocPackage | 1 | ![]() | |
JavadocType | 3 | ![]() | |
JavadocVariable | 28 | ![]() | |
misc | FinalParameters | 15 | ![]() |
NewlineAtEndOfFile | 2 | ![]() | |
sizes | LineLength | 28 | ![]() |
whitespace | OperatorWrap | 1 | ![]() |
ParenPad | 8 | ![]() | |
WhitespaceAround | 10 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 46 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 48 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 51 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 52 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 54 |
![]() |
design | VisibilityModifier | Variable 'config' must be private and have accessor methods. | 54 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 56 |
![]() |
misc | FinalParameters | Parameter config should be final. | 56 |
![]() |
coding | HiddenField | 'config' hides a field. | 56 |
![]() |
design | DesignForExtension | Class 'CassandraClient' looks like designed for extension (can be subclassed), but the method 'start' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraClient' final or making the method 'start' static/final/abstract/empty, or adding allowed annotation for the method. | 61 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 61 |
![]() |
whitespace | WhitespaceAround | '*' is not preceded with whitespace. | 73 |
![]() |
whitespace | WhitespaceAround | '*' is not followed by whitespace. | 73 |
![]() |
coding | MagicNumber | '10' is a magic number. | 73 |
![]() |
whitespace | WhitespaceAround | '*' is not preceded with whitespace. | 74 |
![]() |
whitespace | WhitespaceAround | '*' is not followed by whitespace. | 74 |
![]() |
coding | MagicNumber | '10' is a magic number. | 74 |
![]() |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 77 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 77 |
![]() |
coding | SimplifyBooleanExpression | Expression can be simplified. | 77 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 107). | 88 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 101). | 91 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 94). | 107 |
![]() |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 112 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 112 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 112 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 124 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 124 |
![]() |
whitespace | WhitespaceAround | 'catch' is not followed by whitespace. | 128 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 128 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 128 |
![]() |
whitespace | WhitespaceAround | 'catch' is not followed by whitespace. | 135 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 135 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 135 |
![]() |
design | DesignForExtension | Class 'CassandraClient' looks like designed for extension (can be subclassed), but the method 'stop' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraClient' final or making the method 'stop' static/final/abstract/empty, or adding allowed annotation for the method. | 144 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 144 |
![]() |
design | DesignForExtension | Class 'CassandraClient' looks like designed for extension (can be subclassed), but the method 'config' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraClient' final or making the method 'config' static/final/abstract/empty, or adding allowed annotation for the method. | 149 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 149 |
![]() |
design | DesignForExtension | Class 'CassandraClient' looks like designed for extension (can be subclassed), but the method 'client' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraClient' final or making the method 'client' static/final/abstract/empty, or adding allowed annotation for the method. | 153 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 153 |
![]() |
design | DesignForExtension | Class 'CassandraClient' looks like designed for extension (can be subclassed), but the method 'cluster' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraClient' final or making the method 'cluster' static/final/abstract/empty, or adding allowed annotation for the method. | 157 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 157 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
misc | NewlineAtEndOfFile | File does not end with a newline. | |
![]() |
imports | UnusedImports | Unused import - com.datastax.driver.core.Cluster. | 29 |
![]() |
imports | UnusedImports | Unused import - com.datastax.driver.core.DataType. | 30 |
![]() |
imports | UnusedImports | Unused import - com.datastax.driver.core.KeyspaceMetadata. | 31 |
![]() |
imports | UnusedImports | Unused import - com.datastax.driver.core.Metadata. | 32 |
![]() |
imports | UnusedImports | Unused import - com.datastax.driver.core.Session. | 35 |
![]() |
imports | UnusedImports | Unused import - com.datastax.driver.core.TableMetadata. | 36 |
![]() |
imports | UnusedImports | Unused import - com.datastax.driver.core.schemabuilder.SchemaBuilder. | 38 |
![]() |
imports | UnusedImports | Unused import - org.apache.commons.lang3.StringUtils. | 42 |
![]() |
imports | UnusedImports | Unused import - java.util.HashMap. | 49 |
![]() |
imports | UnusedImports | Unused import - java.util.Map. | 51 |
![]() |
imports | UnusedImports | Unused import - java.util.Objects. | 52 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 92). | 69 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 69 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 71 |
![]() |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 71 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 73 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 75 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 82). | 76 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 76 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 78 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 79 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 81 |
![]() |
design | VisibilityModifier | Variable 'rowIterator' must be private and have accessor methods. | 81 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 83 |
![]() |
design | VisibilityModifier | Variable 'lock' must be private and have accessor methods. | 83 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 95). | 86 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 83). | 90 |
![]() |
misc | FinalParameters | Parameter config should be final. | 97 |
![]() |
coding | HiddenField | 'config' hides a field. | 97 |
![]() |
misc | FinalParameters | Parameter persistQueue should be final. | 105 |
![]() |
coding | HiddenField | 'persistQueue' hides a field. | 105 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 83). | 107 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' looks like designed for extension (can be subclassed), but the method 'setPersistQueue' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraPersistReader' final or making the method 'setPersistQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 111 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 111 |
![]() |
misc | FinalParameters | Parameter persistQueue should be final. | 111 |
![]() |
coding | HiddenField | 'persistQueue' hides a field. | 111 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' looks like designed for extension (can be subclassed), but the method 'getPersistQueue' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraPersistReader' final or making the method 'getPersistQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 115 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 115 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 119 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 122 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 127 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 128 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 150 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' looks like designed for extension (can be subclassed), but the method 'prepareDatum' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraPersistReader' final or making the method 'prepareDatum' static/final/abstract/empty, or adding allowed annotation for the method. | 155 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 155 |
![]() |
misc | FinalParameters | Parameter row should be final. | 155 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 169 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'readAll' static/final/abstract/empty, or adding allowed annotation for the method. | 175 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'startStream' static/final/abstract/empty, or adding allowed annotation for the method. | 189 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 192 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 207 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 224 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 224 |
![]() |
misc | FinalParameters | Parameter entry should be final. | 224 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 238 |
![]() |
misc | FinalParameters | Parameter sequence should be final. | 239 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 243 |
![]() |
misc | FinalParameters | Parameter start should be final. | 244 |
![]() |
misc | FinalParameters | Parameter end should be final. | 244 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReader' 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 'CassandraPersistReader' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 248 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 253 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 82). | 254 |
![]() |
coding | MagicNumber | '10000' is a magic number. | 254 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 257 |
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 263 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 265 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 267 |
![]() |
misc | FinalParameters | Parameter reader should be final. | 267 |
![]() |
coding | HiddenField | 'reader' hides a field. | 267 |
![]() |
design | DesignForExtension | Class 'CassandraPersistReaderTask' 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 'CassandraPersistReaderTask' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 271 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
misc | NewlineAtEndOfFile | File does not end with a newline. | |
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
imports | UnusedImports | Unused import - com.datastax.driver.core.Cluster. | 30 |
![]() |
imports | UnusedImports | Unused import - org.apache.commons.lang3.StringUtils. | 42 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 101). | 65 |
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 65 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 93). | 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 'persistQueue' must be private and have accessor methods. | 73 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 75 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 76 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 76 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 102). | 77 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 77 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 79 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 80 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 82 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 84 |
![]() |
design | VisibilityModifier | Variable 'insertStatement' must be private and have accessor methods. | 84 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 86 |
![]() |
design | VisibilityModifier | Variable 'insertBatch' must be private and have accessor methods. | 86 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 88 |
![]() |
design | VisibilityModifier | Variable 'lock' must be private and have accessor methods. | 88 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 90 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 82). | 92 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 95 |
![]() |
misc | FinalParameters | Parameter config should be final. | 95 |
![]() |
coding | HiddenField | 'config' hides a field. | 95 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' looks like designed for extension (can be subclassed), but the method 'setPersistQueue' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraPersistWriter' final or making the method 'setPersistQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 99 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 99 |
![]() |
misc | FinalParameters | Parameter persistQueue should be final. | 99 |
![]() |
coding | HiddenField | 'persistQueue' hides a field. | 99 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' looks like designed for extension (can be subclassed), but the method 'getPersistQueue' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraPersistWriter' final or making the method 'getPersistQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 103 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 103 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' 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 'CassandraPersistWriter' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 107 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' 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 'CassandraPersistWriter' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 112 |
![]() |
misc | FinalParameters | Parameter streamsDatum should be final. | 113 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 87). | 119 |
![]() |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 124 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 128 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 88). | 131 |
![]() |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 141 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 145 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 88). | 148 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' 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 'CassandraPersistWriter' final or making the method 'flush' static/final/abstract/empty, or adding allowed annotation for the method. | 156 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 84). | 159 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' 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 'CassandraPersistWriter' final or making the method 'close' static/final/abstract/empty, or adding allowed annotation for the method. | 173 |
![]() |
coding | MagicNumber | '15' is a magic number. | 220 |
![]() |
coding | MagicNumber | '15' is a magic number. | 223 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' 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 'CassandraPersistWriter' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 235 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' 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 'CassandraPersistWriter' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 254 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 255 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' 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 'CassandraPersistWriter' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 260 |
![]() |
design | DesignForExtension | Class 'CassandraPersistWriter' looks like designed for extension (can be subclassed), but the method 'flushIfNecessary' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CassandraPersistWriter' final or making the method 'flushIfNecessary' static/final/abstract/empty, or adding allowed annotation for the method. | 265 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 265 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 115). | 267 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 103). | 269 |
![]() |
coding | MagicNumber | '100' is a magic number. | 269 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 278 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 282 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 89). | 285 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 91). | 290 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 117). | 301 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 100). | 303 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 97). | 304 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 310 |