The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | EmptyStatement | 2 |  Error |
| HiddenField | 6 | Error | |
| MagicNumber | 3 | Error | |
| design | DesignForExtension | 25 | Error |
| FinalClass | 2 | Error | |
| HideUtilityClassConstructor | 1 | Error | |
| VisibilityModifier | 9 | Error | |
| imports | UnusedImports
|
1 | Error |
| javadoc | JavadocMethod | 30 | Error |
| JavadocPackage | 3 | Error | |
| JavadocStyle | 2 | Error | |
| JavadocType | 3 | Error | |
| JavadocVariable | 55 | Error | |
| misc | FinalParameters | 47 | Error |
| NewlineAtEndOfFile | 1 | Error | |
| modifier | ModifierOrder | 3 | Error |
| naming | ConstantName | 10 | Error |
| StaticVariableName | 2 | Error | |
| sizes | LineLength | 97 | Error |
| whitespace | OperatorWrap | 12 | Error |
| ParenPad | 69 | Error | |
| WhitespaceAfter | 5 | Error | |
| WhitespaceAround | 47 | Error |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 44 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 49 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 93). | 51 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 51 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 104). | 53 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 53 |
| Error |
naming | ConstantName | Name 'getVertexLongIdStatementTemplate' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 53 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 101). | 54 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 54 |
| Error |
naming | ConstantName | Name 'getVertexStringIdStatementTemplate' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 54 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 55 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 55 |
| Error |
naming | ConstantName | Name 'getVerticesLabelIdStatementTemplate' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 55 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 87). | 57 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 57 |
| Error |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 57 |
| Error |
naming | ConstantName | Name 'createVertexStatementTemplate' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 57 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 57 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 57 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 58 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 58 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 59 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 59 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 62 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 62 |
| Error |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 62 |
| Error |
naming | ConstantName | Name 'mergeVertexStatementTemplate' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 62 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 62 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 62 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 63 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 63 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 64 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 64 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 128). | 67 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
| Error |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 67 |
| Error |
naming | ConstantName | Name 'createEdgeStatementTemplate' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 67 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 67 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 67 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 68 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 68 |
| Error |
design | DesignForExtension | Class 'CypherQueryGraphHelper' looks like designed for extension (can be subclassed), but the method 'getVertexRequest' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'CypherQueryGraphHelper' final or making the method 'getVertexRequest' static/final/abstract/empty, or adding allowed annotation for the method. | 71 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 71 |
| Error |
misc | FinalParameters | Parameter streamsId should be final. | 71 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 76 |
| Error |
misc | FinalParameters | Parameter vertexId should be final. | 88 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 93 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 106 |
| Error |
misc | FinalParameters | Parameter activityObject should be final. | 106 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 116 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 116 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 123 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 99). | 125 |
| Error |
misc | FinalParameters | Parameter labelId should be final. | 137 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 141 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 94). | 153 |
| Error |
misc | FinalParameters | Parameter activityObject should be final. | 153 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 164 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 164 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 170 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 185 |
| Error |
misc | FinalParameters | Parameter activity should be final. | 185 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 190 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 215 |
| Error |
misc | FinalParameters | Parameter activity should be final. | 215 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 220 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 87). | 245 |
| Error |
misc | FinalParameters | Parameter map should be final. | 245 |
| Error |
misc | FinalParameters | Parameter symbol should be final. | 245 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'symbol'. | 245 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 247 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 247 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 248 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 248 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 248 |
| Error |
whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 249 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 117). | 250 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 87). | 261 |
| Error |
misc | FinalParameters | Parameter map should be final. | 261 |
| Error |
misc | FinalParameters | Parameter symbol should be final. | 261 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'symbol'. | 261 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 263 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 263 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 264 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 264 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 264 |
| Error |
whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 265 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 117). | 266 |
| Error |
misc | FinalParameters | Parameter map should be final. | 277 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 281 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 281 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 282 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 282 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 282 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 92). | 284 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 284 |
| Error |
whitespace | WhitespaceAround | '+' is not followed by whitespace. | 284 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 292 |
| Error |
misc | FinalParameters | Parameter map should be final. | 292 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 296 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 296 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 297 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 297 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 297 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 299 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 299 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 300 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 300 |
| Error |
whitespace | WhitespaceAround | '+' is not followed by whitespace. | 300 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 309 |
| Error |
misc | FinalParameters | Parameter map should be final. | 309 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 313 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 313 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 314 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 314 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 314 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 316 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 316 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 317 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 317 |
| Error |
whitespace | WhitespaceAround | '+' is not followed by whitespace. | 317 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 93). | 318 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 318 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 318 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 125). | 319 |
| Error |
whitespace | WhitespaceAround | '+' is not preceded with whitespace. | 319 |
| Error |
whitespace | WhitespaceAround | '+' is not followed by whitespace. | 319 |
| Error |
misc | FinalParameters | Parameter activityObject should be final. | 333 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 335 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 335 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 102). | 336 |
| Error |
whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 336 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 337 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 337 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 46 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 93). | 48 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 48 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 50 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 52 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 112). | 54 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 54 |
| Error |
misc | FinalParameters | Parameter entry should be final. | 54 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 71 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 83 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 83 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 88 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 84). | 98 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 98 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 102 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 93). | 113 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 113 |
| Error |
misc | FinalParameters | Parameter activity should be final. | 113 |
| Error |
coding | EmptyStatement | Empty statement. | 114 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 82). | 125 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 82). | 130 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 91). | 137 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 137 |
| Error |
misc | FinalParameters | Parameter activity should be final. | 137 |
| Error |
coding | EmptyStatement | Empty statement. | 138 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 108). | 143 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 106). | 144 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 108). | 151 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 106). | 152 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 98). | 162 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 162 |
| Error |
misc | FinalParameters | Parameter activityObject should be final. | 162 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 101). | 163 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
imports | UnusedImports | Unused import - com.google.common.base.Preconditions. | 23 |
| Error |
design | FinalClass | Class Neo4jBoltClient should be declared as final. | 39 |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 39 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 41 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 46 |
| Error |
design | VisibilityModifier | Variable 'config' must be private and have accessor methods. | 46 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 48 |
| Error |
misc | FinalParameters | Parameter neo4jConfiguration should be final. | 48 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 136). | 58 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 58 |
| Error |
naming | StaticVariableName | Name 'INSTANCE_MAP' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 58 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 60 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 60 |
| Error |
misc | FinalParameters | Parameter neo4jConfiguration should be final. | 60 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 61 |
| Error |
whitespace | OperatorWrap | '&&' should be on a new line. | 61 |
| Error |
whitespace | OperatorWrap | '&&' should be on a new line. | 62 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 67 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 67 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 67 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 76 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 99). | 79 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 107). | 84 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 84 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 84 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 87). | 85 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 85 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 85 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 88 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 88 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 88 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 105). | 89 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 116). | 91 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 98 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 103 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 107 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
misc | NewlineAtEndOfFile | File does not end with a newline. | |
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 71 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 92). | 73 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 73 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 75 |
| Error |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 75 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 77 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 79 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 82). | 80 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 80 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 82 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 84 |
| Error |
design | VisibilityModifier | Variable 'client' must be private and have accessor methods. | 84 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 91 |
| Error |
design | VisibilityModifier | Variable 'statementResult' must be private and have accessor methods. | 91 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 93 |
| Error |
design | VisibilityModifier | Variable 'lock' must be private and have accessor methods. | 93 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 93). | 96 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 104 |
| Error |
misc | FinalParameters | Parameter config should be final. | 107 |
| Error |
coding | HiddenField | 'config' hides a field. | 107 |
| Error |
misc | FinalParameters | Parameter persistQueue should be final. | 115 |
| Error |
coding | HiddenField | 'persistQueue' hides a field. | 115 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'setPersistQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 120 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 120 |
| Error |
misc | FinalParameters | Parameter persistQueue should be final. | 120 |
| Error |
coding | HiddenField | 'persistQueue' hides a field. | 120 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'getPersistQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 124 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 124 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 128 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 131 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 136 |
| Error |
misc | FinalParameters | Parameter configurationObject should be final. | 137 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 138 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 138 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 138 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 148 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' looks like designed for extension (can be subclassed), but the method 'buildDatum' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Neo4jBoltPersistReader' final or making the method 'buildDatum' static/final/abstract/empty, or adding allowed annotation for the method. | 153 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 153 |
| Error |
misc | FinalParameters | Parameter record should be final. | 153 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 156 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 156 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 156 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 158 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 159 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 159 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 83). | 162 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'readAll' static/final/abstract/empty, or adding allowed annotation for the method. | 169 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 84). | 175 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 93). | 181 |
| Error |
whitespace | WhitespaceAround | 'while' is not followed by whitespace. | 183 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 183 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 186 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 186 |
| Error |
whitespace | WhitespaceAround | 'catch' is not followed by whitespace. | 191 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 194 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 194 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 194 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'startStream' static/final/abstract/empty, or adding allowed annotation for the method. | 202 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 205 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 220 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 237 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 237 |
| Error |
misc | FinalParameters | Parameter entry should be final. | 237 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 251 |
| Error |
misc | FinalParameters | Parameter sequence should be final. | 252 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 256 |
| Error |
misc | FinalParameters | Parameter start should be final. | 257 |
| Error |
misc | FinalParameters | Parameter end should be final. | 257 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReader' 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 'Neo4jBoltPersistReader' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 261 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 266 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 82). | 267 |
| Error |
coding | MagicNumber | '10000' is a magic number. | 267 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 270 |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 274 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 276 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 278 |
| Error |
misc | FinalParameters | Parameter reader should be final. | 278 |
| Error |
coding | HiddenField | 'reader' hides a field. | 278 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistReaderTask' 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 'Neo4jBoltPersistReaderTask' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 282 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 288 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 288 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 288 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 91). | 298 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 298 |
| Error |
design | VisibilityModifier | Variable 'neo4jObjectNodeFunction' must be private and have accessor methods. | 298 |
| Error |
misc | FinalParameters | Parameter value should be final. | 302 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 312 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 84). | 321 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 43 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 45 |
| Error |
design | VisibilityModifier | Variable 'client' must be private and have accessor methods. | 45 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 93). | 47 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 47 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 49 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 51 |
| Error |
misc | FinalParameters | Parameter config should be final. | 51 |
| Error |
coding | HiddenField | 'config' hides a field. | 51 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistWriter' 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 'Neo4jBoltPersistWriter' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 56 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistWriter' 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 'Neo4jBoltPersistWriter' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 61 |
| Error |
misc | FinalParameters | Parameter configurationObject should be final. | 62 |
| Error |
design | DesignForExtension | Class 'Neo4jBoltPersistWriter' 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 'Neo4jBoltPersistWriter' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 71 |
| Error |
misc | FinalParameters | Parameter entry should be final. | 72 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 80 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 80 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 80 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 106). | 81 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 81 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 81 |
| Error |
whitespace | WhitespaceAround | 'catch' is not followed by whitespace. | 85 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 85 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 85 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 88 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 88 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 88 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
design | FinalClass | Class Neo4jHttpClient should be declared as final. | 33 |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 33 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 35 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 38 |
| Error |
design | VisibilityModifier | Variable 'config' must be private and have accessor methods. | 38 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 40 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 42 |
| Error |
misc | FinalParameters | Parameter neo4jConfiguration should be final. | 42 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 136). | 52 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 52 |
| Error |
naming | StaticVariableName | Name 'INSTANCE_MAP' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 52 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 54 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 54 |
| Error |
misc | FinalParameters | Parameter neo4jConfiguration should be final. | 54 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 55 |
| Error |
whitespace | OperatorWrap | '&&' should be on a new line. | 55 |
| Error |
whitespace | OperatorWrap | '&&' should be on a new line. | 56 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 61 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 61 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 61 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 70 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 101). | 73 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 81 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 85 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 89 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 38 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 91). | 40 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 40 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 42 |
| Error |
naming | ConstantName | Name 'statementKey' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 42 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 43 |
| Error |
naming | ConstantName | Name 'queryKey' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 43 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
| Error |
naming | ConstantName | Name 'paramsKey' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 44 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 45 |
| Error |
naming | ConstantName | Name 'propsKey' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 45 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 53 |
| Error |
misc | FinalParameters | Parameter queryPlusParameters should be final. | 53 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 96). | 64 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 64 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 64 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 97). | 65 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 80 |
| Error |
misc | FinalParameters | Parameter queryPlusParameters should be final. | 80 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 96). | 91 |
| Error |
whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 91 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 91 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 97). | 92 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
sizes | LineLength | Line is longer than 80 characters (found 96). | 62 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 90). | 64 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 64 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 93). | 66 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 66 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 68 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 70 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 71 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 73 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 87). | 76 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 142). | 79 |
| Error |
misc | FinalParameters | Parameter configuration should be final. | 86 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 180). | 87 |
| Error |
whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 87 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 95 |
| Error |
misc | FinalParameters | Parameter uri should be final. | 100 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 84). | 104 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 100). | 109 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 109). | 120 |
| Error |
misc | FinalParameters | Parameter jsonNode should be final. | 125 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 99). | 130 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 132 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 132 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 105). | 142 |
| Error |
design | DesignForExtension | Class 'Neo4jHttpPersistReader' 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 'Neo4jHttpPersistReader' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 151 |
| Error |
design | DesignForExtension | Class 'Neo4jHttpPersistReader' 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 'Neo4jHttpPersistReader' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 156 |
| Error |
misc | FinalParameters | Parameter configurationObject should be final. | 157 |
| Error |
design | DesignForExtension | Class 'Neo4jHttpPersistReader' 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 'Neo4jHttpPersistReader' final or making the method 'readAll' static/final/abstract/empty, or adding allowed annotation for the method. | 169 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
sizes | LineLength | Line is longer than 80 characters (found 84). | 53 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 90). | 58 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 58 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 93). | 60 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 60 |
| 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 | JavadocVariable | Missing a Javadoc comment. | 66 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 68 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 70 |
| Error |
design | VisibilityModifier | Variable 'lock' must be private and have accessor methods. | 70 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 90). | 73 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 136). | 76 |
| Error |
misc | FinalParameters | Parameter configuration should be final. | 83 |
| Error |
coding | HiddenField | 'configuration' hides a field. | 83 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 190). | 84 |
| Error |
whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 84 |
| Error |
design | DesignForExtension | Class 'Neo4jHttpPersistWriter' looks like designed for extension (can be subclassed), but the method 'preparePayload' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Neo4jHttpPersistWriter' final or making the method 'preparePayload' static/final/abstract/empty, or adding allowed annotation for the method. | 89 |
| Error |
misc | FinalParameters | Parameter entry should be final. | 90 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 99). | 92 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 97 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 97 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 97 |
| Error |
design | DesignForExtension | Class 'Neo4jHttpPersistWriter' looks like designed for extension (can be subclassed), but the method 'executePost' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Neo4jHttpPersistWriter' final or making the method 'executePost' static/final/abstract/empty, or adding allowed annotation for the method. | 106 |
| Error |
misc | FinalParameters | Parameter httpPost should be final. | 107 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 129). | 119 |
| Error |
coding | MagicNumber | '200' is a magic number. | 119 |
| Error |
coding | MagicNumber | '201' is a magic number. | 119 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 128). | 123 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 124 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 92). | 136 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 99). | 138 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 141 |
| Error |
design | DesignForExtension | Class 'Neo4jHttpPersistWriter' 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 'Neo4jHttpPersistWriter' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 151 |
| Error |
misc | FinalParameters | Parameter configurationObject should be final. | 152 |
| Error |
design | DesignForExtension | Class 'Neo4jHttpPersistWriter' 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 'Neo4jHttpPersistWriter' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 164 |
Info
Warnings