The following document contains the results of Checkstyle. 
| Rules | Violations | Severity |
|---|---|---|
JavadocPackage
| 1 | |
| NewlineAtEndOfFile | 1 | |
| Translation | 0 | |
| FileLength | 0 | |
FileTabCharacter
| 0 | |
RegexpSingleline
| 1 | |
| JavadocMethod | 29 | |
| JavadocType | 3 | |
| JavadocVariable | 34 | |
| JavadocStyle | 7 | |
| ConstantName | 4 | |
| LocalFinalVariableName | 0 | |
| LocalVariableName | 0 | |
| MemberName | 0 | |
| MethodName | 0 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 0 | |
| TypeName | 0 | |
| AvoidStarImport | 0 | |
| IllegalImport | 0 | |
| RedundantImport | 0 | |
| UnusedImports | 0 | |
| LineLength | 44 | |
| MethodLength | 0 | |
| ParameterNumber | 1 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 0 | |
| NoWhitespaceAfter | 0 | |
| NoWhitespaceBefore | 0 | |
| OperatorWrap | 24 | |
| ParenPad | 0 | |
| TypecastParenPad | 0 | |
| WhitespaceAfter | 1 | |
| WhitespaceAround | 3 | |
| ModifierOrder | 0 | |
| RedundantModifier | 0 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 0 | |
| LeftCurly | 49 | |
| NeedBraces | 0 | |
| RightCurly | 14 | |
| AvoidInlineConditionals | 11 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
| HiddenField | 14 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MagicNumber | 2 | |
| MissingSwitchDefault | 0 | |
| RedundantThrows | 0 | |
| SimplifyBooleanExpression | 0 | |
| SimplifyBooleanReturn | 0 | |
| DesignForExtension | 31 | |
| FinalClass | 0 | |
| HideUtilityClassConstructor | 0 | |
| InterfaceIsType | 0 | |
| VisibilityModifier | 4 | |
| ArrayTypeStyle | 0 | |
| FinalParameters | 0 | |
| TodoComment | 1 | |
| UpperEll | 0 |
| Violation | Message | Line |
|---|---|---|
| Missing package-info.java file. | 0 | |
| Missing a Javadoc comment. | 22 | |
| '{' should be on the previous line. | 23 | |
| Missing a Javadoc comment. | 24 | |
| Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 24 | |
| Missing a Javadoc comment. | 26 | |
| Variable 'host' must be private and have accessor methods. | 26 | |
| Missing a Javadoc comment. | 27 | |
| Variable 'port' must be private and have accessor methods. | 27 | |
| Missing a Javadoc comment. | 28 | |
| Variable 'url' must be private and have accessor methods. | 28 | |
| Missing a Javadoc comment. | 30 | |
| 'host' hides a field. | 30 | |
| 'port' hides a field. | 30 | |
| '{' should be on the previous line. | 31 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 28 | |
| '{' should be on the previous line. | 29 | |
| Missing a Javadoc comment. | 30 | |
| Missing a Javadoc comment. | 31 | |
| Line is longer than 80 characters. | 33 | |
| Missing a Javadoc comment. | 33 | |
| Missing a Javadoc comment. | 34 | |
| Missing a Javadoc comment. | 36 | |
| '{' should be on the previous line. | 37 | |
| Line is longer than 80 characters. | 41 | |
| Missing a Javadoc comment. | 41 | |
| Line is longer than 80 characters. | 47 | |
| '{' should be on the previous line. | 51 | |
| First sentence should end with a period. | 57 | |
| Method 'refreshSchemataCache' is not designed for extension - needs to be abstract, final or empty. | 62 | |
| '{' should be on the previous line. | 63 | |
| Line is longer than 80 characters. | 72 | |
| ',' is not followed by whitespace. | 72 | |
| '}' should be on the same line. | 81 | |
| '}' should be on the same line. | 85 | |
| Line is longer than 80 characters. | 87 | |
| Line is longer than 80 characters. | 92 | |
| Method 'getSchema' is not designed for extension - needs to be abstract, final or empty. | 98 | |
| '{' should be on the previous line. | 99 | |
| Method 'getSchemata' is not designed for extension - needs to be abstract, final or empty. | 109 | |
| Missing a Javadoc comment. | 109 | |
| '{' should be on the previous line. | 110 | |
| First sentence should end with a period. | 114 | |
| Method 'close' is not designed for extension - needs to be abstract, final or empty. | 117 | |
| '{' should be on the previous line. | 118 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 32 | |
| '{' should be on the previous line. | 33 | |
| Missing a Javadoc comment. | 34 | |
| Name 'mapper' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 34 | |
| Missing a Javadoc comment. | 35 | |
| Variable 'client' must be private and have accessor methods. | 35 | |
| Missing a Javadoc comment. | 37 | |
| '{' should be on the previous line. | 38 | |
| Extra HTML tag found: </p> | 45 | |
| Method 'getSchema' is not designed for extension - needs to be abstract, final or empty. | 60 | |
| '{' should be on the previous line. | 61 | |
| Line is longer than 80 characters. | 63 | |
| Line is longer than 80 characters. | 66 | |
| '200' is a magic number. | 68 | |
| '}' should be on the same line. | 75 | |
| '{' should be on the previous line. | 83 | |
| '}' should be on the same line. | 87 | |
| Line is longer than 80 characters. | 97 | |
| Method 'getSchemata' is not designed for extension - needs to be abstract, final or empty. | 102 | |
| '{' should be on the previous line. | 103 | |
| Line is longer than 80 characters. | 105 | |
| Line is longer than 80 characters. | 108 | |
| '200' is a magic number. | 110 | |
| Line is longer than 80 characters. | 116 | |
| Line is longer than 80 characters. | 117 | |
| '}' should be on the same line. | 121 | |
| '{' should be on the previous line. | 129 | |
| '}' should be on the same line. | 133 | |
| Line is longer than 80 characters. | 140 | |
| Missing a Javadoc comment. | 141 | |
| '{' should be on the previous line. | 142 | |
| Line is longer than 80 characters. | 145 | |
| First sentence should end with a period. | 150 | |
| Method 'close' is not designed for extension - needs to be abstract, final or empty. | 153 | |
| '{' should be on the previous line. | 154 | |
| Missing a Javadoc comment. | 158 | |
| '{' should be on the previous line. | 159 | |
| '}' should be on the same line. | 163 | |
| Line is longer than 80 characters. | 165 |
| Violation | Message | Line |
|---|---|---|
| File does not end with a newline. | 0 | |
| '{' should be on the previous line. | 41 | |
| Missing a Javadoc comment. | 42 | |
| Name 'mapper' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 42 | |
| Missing a Javadoc comment. | 44 | |
| Missing a Javadoc comment. | 45 | |
| Line is longer than 80 characters. | 46 | |
| Missing a Javadoc comment. | 46 | |
| Missing a Javadoc comment. | 48 | |
| Missing a Javadoc comment. | 49 | |
| Missing a Javadoc comment. | 50 | |
| First sentence should end with a period. | 52 | |
| 'name' hides a field. | 104 | |
| Line is longer than 80 characters. | 105 | |
| 'sinkAddInfo' hides a field. | 106 | |
| '{' should be on the previous line. | 108 | |
| Line is longer than 80 characters. | 119 | |
| 'name' hides a field. | 119 | |
| Line is longer than 80 characters. | 127 | |
| Missing a Javadoc comment. | 127 | |
| Method 'toMap' is not designed for extension - needs to be abstract, final or empty. | 132 | |
| Missing a Javadoc comment. | 132 | |
| '}' is not preceded with whitespace. | 133 | |
| '{' should be on the previous line. | 135 | |
| Line is longer than 80 characters. | 139 | |
| Avoid inline conditionals. | 139 | |
| Line is longer than 80 characters. | 144 | |
| Method 'addThriftField' is not designed for extension - needs to be abstract, final or empty. | 148 | |
| '{' should be on the previous line. | 149 | |
| Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 153 | |
| Missing a Javadoc comment. | 153 | |
| '{' should be on the previous line. | 154 | |
| Method 'getSchema' is not designed for extension - needs to be abstract, final or empty. | 164 | |
| '{' should be on the previous line. | 165 | |
| Line is longer than 80 characters. | 166 | |
| Line is longer than 80 characters. | 171 | |
| Method 'setSinkAddInfo' is not designed for extension - needs to be abstract, final or empty. | 180 | |
| Missing a Javadoc comment. | 180 | |
| 'sinkAddInfo' hides a field. | 180 | |
| '{' should be on the previous line. | 181 | |
| Method 'getFieldByPosition' is not designed for extension - needs to be abstract, final or empty. | 191 | |
| '{' should be on the previous line. | 192 | |
| Method 'getFieldByName' is not designed for extension - needs to be abstract, final or empty. | 202 | |
| 'name' hides a field. | 202 | |
| '{' should be on the previous line. | 203 | |
| Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 213 | |
| '{' should be on the previous line. | 215 | |
| '}' should be on the same line. | 218 | |
| '+' should be on a new line. | 220 | |
| '+' should be on a new line. | 221 | |
| '+' should be on a new line. | 222 | |
| '}' should be on the same line. | 224 | |
| '+' should be on a new line. | 226 | |
| '+' should be on a new line. | 227 | |
| '+' should be on a new line. | 228 | |
| Method 'toJSONBytes' is not designed for extension - needs to be abstract, final or empty. | 233 | |
| Missing a Javadoc comment. | 233 | |
| '{' should be on the previous line. | 234 | |
| Expected an @return tag. | 241 | |
| Method 'toJSON' is not designed for extension - needs to be abstract, final or empty. | 241 | |
| Expected @throws tag for 'IOException'. | 242 | |
| '{' should be on the previous line. | 243 | |
| Line has trailing spaces. | 244 | |
| Line is longer than 80 characters. | 246 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 33 | |
| '{' should be on the previous line. | 38 | |
| Missing a Javadoc comment. | 39 | |
| Name 'mapper' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 39 | |
| Missing a Javadoc comment. | 41 | |
| Missing a Javadoc comment. | 42 | |
| Missing a Javadoc comment. | 43 | |
| Missing a Javadoc comment. | 44 | |
| Line is longer than 80 characters. | 47 | |
| Line is longer than 80 characters. | 49 | |
| Missing a Javadoc comment. | 52 | |
| Missing a Javadoc comment. | 53 | |
| Missing a Javadoc comment. | 54 | |
| Missing a Javadoc comment. | 55 | |
| Missing a Javadoc comment. | 56 | |
| Missing a Javadoc comment. | 58 | |
| Missing a Javadoc comment. | 59 | |
| Missing a Javadoc comment. | 60 | |
| First sentence should end with a period. | 62 | |
| Line is longer than 80 characters. | 78 | |
| 'description' hides a field. | 89 | |
| 'sql' hides a field. | 90 | |
| '{' should be on the previous line. | 92 | |
| Line is longer than 80 characters. | 93 | |
| Avoid inline conditionals. | 93 | |
| Avoid inline conditionals. | 93 | |
| Avoid inline conditionals. | 93 | |
| Avoid inline conditionals. | 93 | |
| Line is longer than 80 characters. | 100 | |
| More than 7 parameters. | 108 | |
| 'description' hides a field. | 112 | |
| '{' should be on the previous line. | 118 | |
| Line is longer than 80 characters. | 120 | |
| Line is longer than 80 characters. | 123 | |
| Line is longer than 80 characters. | 124 | |
| Missing a Javadoc comment. | 134 | |
| '{' should be on the previous line. | 135 | |
| Line is longer than 80 characters. | 136 | |
| Line is longer than 80 characters. | 139 | |
| Missing a Javadoc comment. | 139 | |
| Method 'toMap' is not designed for extension - needs to be abstract, final or empty. | 144 | |
| Missing a Javadoc comment. | 144 | |
| '}' is not preceded with whitespace. | 145 | |
| '{' should be on the previous line. | 147 | |
| Line is longer than 80 characters. | 152 | |
| Avoid inline conditionals. | 152 | |
| Avoid inline conditionals. | 153 | |
| First sentence should end with a period. | 157 | |
| '{' should be on the previous line. | 161 | |
| Missing a Javadoc comment. | 162 | |
| Missing a Javadoc comment. | 163 | |
| Missing a Javadoc comment. | 164 | |
| Missing a Javadoc comment. | 165 | |
| Missing a Javadoc comment. | 167 | |
| 'type' hides a field. | 169 | |
| 'length' hides a field. | 170 | |
| 'scale' hides a field. | 171 | |
| Line is longer than 80 characters. | 172 | |
| 'precision' hides a field. | 172 | |
| Method 'toMap' is not designed for extension - needs to be abstract, final or empty. | 181 | |
| Missing a Javadoc comment. | 181 | |
| '}' is not preceded with whitespace. | 182 | |
| '{' should be on the previous line. | 184 | |
| Line is longer than 80 characters. | 186 | |
| Avoid inline conditionals. | 186 | |
| Line is longer than 80 characters. | 187 | |
| Avoid inline conditionals. | 187 | |
| Line is longer than 80 characters. | 188 | |
| Avoid inline conditionals. | 188 | |
| Line is longer than 80 characters. | 189 | |
| Avoid inline conditionals. | 189 | |
| Method 'getType' is not designed for extension - needs to be abstract, final or empty. | 193 | |
| Missing a Javadoc comment. | 193 | |
| '{' should be on the previous line. | 194 | |
| Method 'getLength' is not designed for extension - needs to be abstract, final or empty. | 198 | |
| Missing a Javadoc comment. | 198 | |
| '{' should be on the previous line. | 199 | |
| Method 'getScale' is not designed for extension - needs to be abstract, final or empty. | 203 | |
| Missing a Javadoc comment. | 203 | |
| '{' should be on the previous line. | 204 | |
| Method 'getPrecision' is not designed for extension - needs to be abstract, final or empty. | 208 | |
| Missing a Javadoc comment. | 208 | |
| '{' should be on the previous line. | 209 | |
| Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 214 | |
| Missing a Javadoc comment. | 214 | |
| '{' should be on the previous line. | 215 | |
| Method 'getType' is not designed for extension - needs to be abstract, final or empty. | 219 | |
| Missing a Javadoc comment. | 219 | |
| '{' should be on the previous line. | 220 | |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 224 | |
| Missing a Javadoc comment. | 224 | |
| '{' should be on the previous line. | 225 | |
| Method 'getSql' is not designed for extension - needs to be abstract, final or empty. | 229 | |
| Missing a Javadoc comment. | 229 | |
| '{' should be on the previous line. | 230 | |
| Method 'getDescription' is not designed for extension - needs to be abstract, final or empty. | 234 | |
| Missing a Javadoc comment. | 234 | |
| '{' should be on the previous line. | 235 | |
| Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 239 | |
| '{' should be on the previous line. | 241 | |
| '}' should be on the same line. | 244 | |
| '+' should be on a new line. | 246 | |
| '+' should be on a new line. | 247 | |
| '+' should be on a new line. | 248 | |
| '+' should be on a new line. | 249 | |
| '+' should be on a new line. | 250 | |
| '+' should be on a new line. | 251 | |
| '+' should be on a new line. | 252 | |
| '+' should be on a new line. | 253 | |
| '+' should be on a new line. | 254 | |
| '}' should be on the same line. | 256 | |
| '+' should be on a new line. | 258 | |
| '+' should be on a new line. | 259 | |
| '+' should be on a new line. | 260 | |
| '+' should be on a new line. | 261 | |
| '+' should be on a new line. | 262 | |
| '+' should be on a new line. | 263 | |
| '+' should be on a new line. | 264 | |
| '+' should be on a new line. | 265 | |
| '+' should be on a new line. | 266 | |
| Line is longer than 80 characters. | 272 | |
| Unused @throws tag for 'JsonGenerationException'. | 278 | |
| Method 'toJSON' is not designed for extension - needs to be abstract, final or empty. | 280 | |
| '{' should be on the previous line. | 281 | |
| Comment matches to-do format 'TODO:'. | 289 | |
| Method 'getFullSQLType' is not designed for extension - needs to be abstract, final or empty. | 293 | |
| '{' should be on the previous line. | 295 | |
| '}' should be on the same line. | 300 | |
| Line is longer than 80 characters. | 304 | |
| '}' should be on the same line. | 305 | |
| '}' should be on the same line. | 310 |