The following document contains the results of Checkstyle.
Rules | Violations | Severity |
---|---|---|
JavadocPackage
| 0 | Error |
NewlineAtEndOfFile | 0 | Error |
Translation | 0 | Error |
FileLength | 0 | Error |
FileTabCharacter
| 0 | Error |
RegexpSingleline
| 0 | Error |
JavadocMethod | 243 | Error |
JavadocType | 0 | Error |
JavadocVariable | 0 | Error |
JavadocStyle | 1046 | Error |
ConstantName | 0 | Error |
LocalFinalVariableName | 0 | Error |
LocalVariableName | 0 | Error |
MemberName | 0 | Error |
MethodName | 0 | Error |
PackageName | 0 | Error |
ParameterName | 0 | Error |
StaticVariableName | 25 | Error |
TypeName | 0 | Error |
AvoidStarImport | 0 | Error |
IllegalImport | 0 | Error |
RedundantImport | 0 | Error |
UnusedImports | 0 | Error |
LineLength | 0 | Error |
MethodLength | 5 | Error |
ParameterNumber | 0 | Error |
EmptyForIteratorPad | 0 | Error |
MethodParamPad | 2 | Error |
NoWhitespaceAfter | 2 | Error |
NoWhitespaceBefore | 0 | Error |
OperatorWrap | 1010 | Error |
ParenPad | 0 | Error |
TypecastParenPad | 0 | Error |
WhitespaceAfter | 2 | Error |
WhitespaceAround | 0 | Error |
ModifierOrder | 0 | Error |
RedundantModifier | 0 | Error |
AvoidNestedBlocks | 0 | Error |
EmptyBlock | 0 | Error |
LeftCurly | 0 | Error |
NeedBraces | 0 | Error |
RightCurly | 0 | Error |
AvoidInlineConditionals | 28 | Error |
DoubleCheckedLocking | 0 | Error |
EmptyStatement | 0 | Error |
EqualsHashCode | 0 | Error |
HiddenField | 185 | Error |
IllegalInstantiation | 0 | Error |
InnerAssignment | 0 | Error |
MagicNumber | 119 | Error |
MissingSwitchDefault | 0 | Error |
RedundantThrows | 0 | Error |
SimplifyBooleanExpression | 0 | Error |
SimplifyBooleanReturn | 0 | Error |
DesignForExtension | 749 | Error |
FinalClass | 14 | Error |
HideUtilityClassConstructor | 0 | Error |
InterfaceIsType | 0 | Error |
VisibilityModifier | 25 | Error |
ArrayTypeStyle | 0 | Error |
FinalParameters | 1050 | Error |
TodoComment | 3 | Error |
UpperEll | 0 | Error |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 56 | |
First sentence should end with a period. | 58 | |
'+' should be on a new line. | 76 | |
'+' should be on a new line. | 89 | |
'+' should be on a new line. | 90 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 95 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 100 | |
Parameter conf should be final. | 101 | |
'conf' hides a field. | 101 | |
First sentence should end with a period. | 105 | |
Method 'run' is not designed for extension - needs to be abstract, final or empty. | 128 | |
Parameter args should be final. | 129 | |
'+' should be on a new line. | 172 | |
'+' should be on a new line. | 182 | |
':' should be on a new line. | 210 | |
'+' should be on a new line. | 215 | |
'+' should be on a new line. | 219 | |
'100.0f' is a magic number. | 233 | |
Avoid inline conditionals. | 237 | |
Parameter args should be final. | 246 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 33 | |
Method 'getAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 44 | |
Method 'setAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 49 | |
Parameter value should be final. | 50 | |
'value' hides a field. | 50 | |
Method 'reset' is not designed for extension - needs to be abstract, final or empty. | 54 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 28 | |
Parameter value should be final. | 29 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 28 | |
Parameter value should be final. | 29 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 32 | |
Parameter value should be final. | 33 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 37 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 31 | |
Parameter value should be final. | 32 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 36 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 32 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 25 | |
Parameter value should be final. | 26 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 30 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 30 | |
Parameter value should be final. | 31 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 35 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 32 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 32 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 30 | |
Parameter value should be final. | 31 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 35 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 32 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 32 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 33 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 30 | |
Parameter value should be final. | 31 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 35 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 32 |
Violation | Message | Line |
---|---|---|
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Parameter value should be final. | 28 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 32 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 26 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 33 | |
Parameter messages should be final. | 34 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 27 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 34 | |
Parameter messages should be final. | 35 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 45 | |
First sentence should end with a period. | 47 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 50 | |
Parameter messages should be final. | 51 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 55 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 60 | |
Parameter conf should be final. | 61 | |
'conf' hides a field. | 61 | |
'+' should be on a new line. | 114 | |
'||' should be on a new line. | 121 | |
'+' should be on a new line. | 127 | |
'100.0f' is a magic number. | 130 |
Violation | Message | Line |
---|---|---|
Class PageRankComputation should be declared as final. | 28 | |
First sentence should end with a period. | 29 | |
Parameter vertex should be final. | 45 | |
Parameter messages should be final. | 46 | |
'0.15f' is a magic number. | 53 | |
'0.85f' is a magic number. | 53 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 46 | |
First sentence should end with a period. | 49 | |
First sentence should end with a period. | 52 | |
First sentence should end with a period. | 54 | |
First sentence should end with a period. | 57 | |
First sentence should end with a period. | 59 | |
First sentence should end with a period. | 62 | |
First sentence should end with a period. | 64 | |
First sentence should end with a period. | 67 | |
First sentence should end with a period. | 69 | |
First sentence should end with a period. | 72 | |
First sentence should end with a period. | 74 | |
First sentence should end with a period. | 76 | |
First sentence should end with a period. | 79 | |
First sentence should end with a period. | 87 | |
First sentence should end with a period. | 90 | |
First sentence should end with a period. | 92 | |
First sentence should end with a period. | 94 | |
First sentence should end with a period. | 96 | |
First sentence should end with a period. | 98 | |
First sentence should end with a period. | 100 | |
First sentence should end with a period. | 102 | |
First sentence should end with a period. | 104 | |
'1000d' is a magic number. | 143 | |
'1024d' is a magic number. | 143 | |
'1024d' is a magic number. | 143 | |
'*' should be on a new line. | 144 | |
'1000d' is a magic number. | 145 | |
'1024d' is a magic number. | 145 | |
'1024d' is a magic number. | 145 | |
'1000d' is a magic number. | 147 | |
'1000d' is a magic number. | 149 | |
'+' should be on a new line. | 159 | |
'+' should be on a new line. | 161 | |
'+' should be on a new line. | 163 | |
'+' should be on a new line. | 165 | |
'+' should be on a new line. | 167 | |
'+' should be on a new line. | 169 | |
'+' should be on a new line. | 171 | |
'+' should be on a new line. | 173 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 233 | |
First sentence should end with a period. | 247 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 252 | |
Parameter messages should be final. | 253 | |
'*' should be on a new line. | 261 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 273 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 278 | |
Parameter conf should be final. | 279 | |
'conf' hides a field. | 279 | |
Method 'run' is not designed for extension - needs to be abstract, final or empty. | 283 | |
Parameter args should be final. | 284 | |
'+' should be on a new line. | 341 | |
'100.0f' is a magic number. | 360 | |
Parameter args should be final. | 402 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 45 | |
First sentence should end with a period. | 48 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 51 | |
Parameter messages should be final. | 52 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 56 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 61 | |
Parameter conf should be final. | 62 | |
'conf' hides a field. | 62 | |
'+' should be on a new line. | 107 | |
'||' should be on a new line. | 114 | |
'+' should be on a new line. | 120 | |
'100.0f' is a magic number. | 124 |
Violation | Message | Line |
---|---|---|
Class ShortestPathsComputation should be declared as final. | 29 | |
Parameter vertex should be final. | 44 | |
'==' should be on a new line. | 46 | |
Parameter vertex should be final. | 58 | |
Parameter messages should be final. | 60 | |
Avoid inline conditionals. | 65 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 18 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 21 | |
First sentence should end with a period. | 25 | |
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 29 | |
First sentence should end with a period. | 31 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 42 | |
First sentence should end with a period. | 45 | |
Parameter conf should be final. | 51 | |
'+' should be on a new line. | 64 | |
'+' should be on a new line. | 65 | |
Method 'getSplits' is not designed for extension - needs to be abstract, final or empty. | 71 | |
Parameter context should be final. | 72 | |
Method 'createRecordReader' is not designed for extension - needs to be abstract, final or empty. | 87 | |
Parameter split should be final. | 89 | |
Parameter context should be final. | 89 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 35 | |
First sentence should end with a period. | 37 | |
Parameter splitIndex should be final. | 51 | |
'splitIndex' hides a field. | 51 | |
Parameter numSplits should be final. | 51 | |
'numSplits' hides a field. | 51 | |
Method 'getLength' is not designed for extension - needs to be abstract, final or empty. | 56 | |
Method 'getLocations' is not designed for extension - needs to be abstract, final or empty. | 61 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 66 | |
Parameter in should be final. | 67 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 72 | |
Parameter out should be final. | 73 | |
Method 'getSplitIndex' is not designed for extension - needs to be abstract, final or empty. | 83 | |
Method 'getNumSplits' is not designed for extension - needs to be abstract, final or empty. | 92 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 96 | |
'+' should be on a new line. | 98 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 37 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 38 | |
Method 'checkOutputSpecs' is not designed for extension - needs to be abstract, final or empty. | 40 | |
Parameter context should be final. | 41 | |
'==' should be on a new line. | 43 | |
'+' should be on a new line. | 45 | |
Method 'getOutputCommitter' is not designed for extension - needs to be abstract, final or empty. | 53 | |
Parameter context should be final. | 54 | |
'==' should be on a new line. | 56 | |
'+' should be on a new line. | 58 | |
Method 'getRecordWriter' is not designed for extension - needs to be abstract, final or empty. | 66 | |
Parameter context should be final. | 67 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 32 | |
First sentence should end with a period. | 34 | |
Avoid inline conditionals. | 47 | |
Parameter inputSplit should be final. | 61 | |
Parameter context should be final. | 61 |
Violation | Message | Line |
---|---|---|
Parameter context should be final. | 35 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 40 | |
Parameter key should be final. | 41 | |
Parameter value should be final. | 41 | |
'+' should be on a new line. | 43 | |
'+' should be on a new line. | 44 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 50 | |
First sentence should end with a period. | 66 | |
Expected @throws tag for 'IOException'. | 72 | |
Expected @throws tag for 'InterruptedException'. | 72 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 57 | |
Expected @throws tag for 'KeeperException'. | 65 | |
Expected @throws tag for 'InterruptedException'. | 65 | |
First sentence should end with a period. | 75 | |
Expected @throws tag for 'IOException'. | 81 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 52 | |
Expected @throws tag for 'IOException'. | 87 | |
Expected @throws tag for 'IOException'. | 97 | |
First sentence should end with a period. | 178 | |
First sentence should end with a period. | 198 |
Violation | Message | Line |
---|---|---|
Parameter context should be final. | 34 | |
Parameter context should be final. | 38 | |
Method 'needsTaskCommit' is not designed for extension - needs to be abstract, final or empty. | 41 | |
Parameter context should be final. | 42 | |
Parameter context should be final. | 48 | |
Parameter context should be final. | 52 | |
Parameter jobContext should be final. | 61 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 21 | |
First sentence should end with a period. | 25 | |
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 29 | |
First sentence should end with a period. | 31 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 83 | |
First sentence should end with a period. | 86 | |
First sentence should end with a period. | 88 | |
Variable 'conf' must be private and have accessor methods. | 89 | |
First sentence should end with a period. | 90 | |
First sentence should end with a period. | 92 | |
First sentence should end with a period. | 94 | |
First sentence should end with a period. | 96 | |
First sentence should end with a period. | 98 | |
First sentence should end with a period. | 100 | |
First sentence should end with a period. | 102 | |
First sentence should end with a period. | 104 | |
First sentence should end with a period. | 106 | |
First sentence should end with a period. | 108 | |
First sentence should end with a period. | 110 | |
First sentence should end with a period. | 122 | |
First sentence should end with a period. | 126 | |
First sentence should end with a period. | 155 | |
First sentence should end with a period. | 161 | |
First sentence should end with a period. | 163 | |
First sentence should end with a period. | 165 | |
First sentence should end with a period. | 180 | |
First sentence should end with a period. | 184 | |
First sentence should end with a period. | 187 | |
Parameter idMessageMap should be final. | 193 | |
Parameter peerConnection should be final. | 194 | |
Parameter isProxy should be final. | 195 | |
'isProxy' hides a field. | 195 | |
First sentence should end with a period. | 230 | |
Parameter peerConnection should be final. | 243 | |
'peerConnection' hides a field. | 243 | |
Parameter context should be final. | 244 | |
'context' hides a field. | 244 | |
':' should be on a new line. | 262 | |
'+' should be on a new line. | 267 | |
'<' should be on a new line. | 278 | |
'+' should be on a new line. | 281 | |
'+' should be on a new line. | 282 | |
'+' should be on a new line. | 292 | |
'||' should be on a new line. | 308 | |
'<' should be on a new line. | 309 | |
':' should be on a new line. | 319 | |
'+' should be on a new line. | 330 | |
'100f' is a magic number. | 335 | |
'/' should be on a new line. | 335 | |
'1000f' is a magic number. | 338 | |
'60f' is a magic number. | 338 | |
'100f' is a magic number. | 340 | |
'-' should be on a new line. | 340 | |
'+' should be on a new line. | 342 | |
'+' should be on a new line. | 343 | |
'+' should be on a new line. | 344 | |
'+' should be on a new line. | 345 | |
'+' should be on a new line. | 346 | |
'+' should be on a new line. | 347 | |
'+' should be on a new line. | 348 | |
'+' should be on a new line. | 349 | |
'+' should be on a new line. | 357 | |
First sentence should end with a period. | 378 | |
Parameter peerConnection should be final. | 389 | |
'peerConnection' hides a field. | 389 | |
Parameter destVertex should be final. | 389 | |
'destVertex' hides a field. | 389 | |
'<' should be on a new line. | 412 | |
'+' should be on a new line. | 415 | |
'+' should be on a new line. | 416 | |
Parameter context should be final. | 446 | |
'context' hides a field. | 446 | |
Parameter service should be final. | 447 | |
'service' hides a field. | 447 | |
Expected @throws tag for 'IOException'. | 448 | |
Expected @throws tag for 'InterruptedException'. | 448 | |
'10' is a magic number. | 491 | |
'+' should be on a new line. | 494 | |
'+' should be on a new line. | 506 | |
'+' should be on a new line. | 507 | |
'+' should be on a new line. | 520 | |
'+' should be on a new line. | 528 | |
'+' should be on a new line. | 536 | |
'+' should be on a new line. | 537 | |
'+' should be on a new line. | 538 | |
Parameter addr should be final. | 549 | |
Parameter destVertex should be final. | 549 | |
Expected @throws tag for 'IOException'. | 560 | |
'jobId' hides a field. | 572 | |
'jobToken' hides a field. | 572 | |
Expected @throws tag for 'IOException'. | 572 | |
Method 'getPort' is not designed for extension - needs to be abstract, final or empty. | 579 | |
Method 'setup' is not designed for extension - needs to be abstract, final or empty. | 583 | |
First sentence should end with a period. | 595 | |
'jobId' hides a field. | 606 | |
'jobToken' hides a field. | 606 | |
Expected @throws tag for 'IOException'. | 607 | |
Expected @throws tag for 'InterruptedException'. | 607 | |
Parameter jobId should be final. | 618 | |
'jobId' hides a field. | 618 | |
Parameter jobToken should be final. | 618 | |
'jobToken' hides a field. | 618 | |
Expected @throws tag for 'IOException'. | 619 | |
Expected @throws tag for 'InterruptedException'. | 619 | |
'+' should be on a new line. | 629 | |
'+' should be on a new line. | 630 | |
Parameter workerInfo should be final. | 648 | |
Parameter jobId should be final. | 649 | |
'jobId' hides a field. | 649 | |
Parameter jobToken should be final. | 650 | |
'jobToken' hides a field. | 650 | |
Expected @throws tag for 'IOException'. | 650 | |
Expected @throws tag for 'InterruptedException'. | 650 | |
'+' should be on a new line. | 652 | |
'+' should be on a new line. | 653 | |
'+' should be on a new line. | 670 | |
'+' should be on a new line. | 671 | |
Parameter protocol should be final. | 694 | |
Parameter clientVersion should be final. | 694 | |
Method 'getProtocolSignature' is not designed for extension - needs to be abstract, final or empty. | 710 | |
Parameter protocol should be final. | 711 | |
Parameter clientVersion should be final. | 712 | |
Parameter clientMethodsHash should be final. | 713 | |
Expected @throws tag for 'IOException'. | 713 | |
Method 'closeConnections' is not designed for extension - needs to be abstract, final or empty. | 718 | |
Parameter vertex should be final. | 733 | |
Parameter msg should be final. | 733 | |
Parameter vertex should be final. | 761 | |
Parameter msgList should be final. | 762 | |
'+' should be on a new line. | 765 | |
Parameter vertexIdMessagesList should be final. | 782 | |
'+' should be on a new line. | 785 | |
Parameter partitionId should be final. | 808 | |
Parameter vertexList should be final. | 809 | |
'+' should be on a new line. | 811 | |
Parameter sourceVertexId should be final. | 818 | |
Parameter targetVertexId should be final. | 818 | |
Parameter edgeValue should be final. | 818 | |
Method 'removeEdge' is not designed for extension - needs to be abstract, final or empty. | 835 | |
Parameter vertexIndex should be final. | 836 | |
Parameter destinationVertexIndex should be final. | 836 | |
'+' should be on a new line. | 838 | |
Parameter vertex should be final. | 854 | |
Method 'removeVertex' is not designed for extension - needs to be abstract, final or empty. | 870 | |
Parameter vertexIndex should be final. | 871 | |
Parameter workerInfo should be final. | 888 | |
Parameter partition should be final. | 889 | |
'+' should be on a new line. | 900 | |
'+' should be on a new line. | 901 | |
Parameter workerInfo should be final. | 933 | |
Parameter partitionId should be final. | 934 | |
'||' should be on a new line. | 945 | |
'+' should be on a new line. | 948 | |
Parameter destVertex should be final. | 962 | |
Parameter destVertex should be final. | 970 | |
Parameter msg should be final. | 970 | |
'+' should be on a new line. | 973 | |
'+' should be on a new line. | 983 | |
'+' should be on a new line. | 995 | |
Parameter destVertex should be final. | 1005 | |
Parameter edge should be final. | 1005 | |
'+' should be on a new line. | 1009 | |
Parameter vertexIndex should be final. | 1018 | |
Parameter destVertexIndex should be final. | 1018 | |
'+' should be on a new line. | 1022 | |
Parameter vertex should be final. | 1031 | |
'+' should be on a new line. | 1035 | |
Method 'removeVertexRequest' is not designed for extension - needs to be abstract, final or empty. | 1043 | |
Parameter vertexIndex should be final. | 1044 | |
'+' should be on a new line. | 1048 | |
Method 'flush' is not designed for extension - needs to be abstract, final or empty. | 1056 | |
'+' should be on a new line. | 1059 | |
'+' should be on a new line. | 1060 | |
'+' should be on a new line. | 1093 | |
'+' should be on a new line. | 1094 | |
Method 'resetMessageCount' is not designed for extension - needs to be abstract, final or empty. | 1099 | |
Method 'prepareSuperstep' is not designed for extension - needs to be abstract, final or empty. | 1106 | |
Method length is 159 lines (max allowed is 150). | 1106 | |
'+' should be on a new line. | 1109 | |
'+' should be on a new line. | 1110 | |
'+' should be on a new line. | 1125 | |
'<' should be on a new line. | 1128 | |
'+' should be on a new line. | 1131 | |
'+' should be on a new line. | 1132 | |
'+' should be on a new line. | 1133 | |
':' should be on a new line. | 1159 | |
'+' should be on a new line. | 1165 | |
'+' should be on a new line. | 1166 | |
'+' should be on a new line. | 1171 | |
'+' should be on a new line. | 1179 | |
'+' should be on a new line. | 1197 | |
'+' should be on a new line. | 1198 | |
'+' should be on a new line. | 1199 | |
'+' should be on a new line. | 1200 | |
'+' should be on a new line. | 1201 | |
'+' should be on a new line. | 1202 | |
'+' should be on a new line. | 1203 | |
'+' should be on a new line. | 1204 | |
'+' should be on a new line. | 1240 | |
'+' should be on a new line. | 1241 | |
'+' should be on a new line. | 1242 | |
'+' should be on a new line. | 1243 | |
'+' should be on a new line. | 1244 | |
'+' should be on a new line. | 1252 | |
'+' should be on a new line. | 1253 | |
Method 'fixPartitionIdToSocketAddrMap' is not designed for extension - needs to be abstract, final or empty. | 1267 | |
'&&' should be on a new line. | 1276 | |
'||' should be on a new line. | 1278 | |
'!=' should be on a new line. | 1279 | |
'+' should be on a new line. | 1282 | |
'+' should be on a new line. | 1283 | |
'+' should be on a new line. | 1284 | |
Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 1301 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 50 | |
Expected @throws tag for 'IOException'. | 64 | |
Expected @throws tag for 'IOException'. | 73 | |
Expected @throws tag for 'IOException'. | 82 | |
Expected @throws tag for 'IOException'. | 91 | |
First sentence should end with a period. | 93 | |
Expected @throws tag for 'IOException'. | 102 | |
First sentence should end with a period. | 104 | |
Expected @throws tag for 'IOException'. | 111 | |
First sentence should end with a period. | 113 | |
Expected @throws tag for 'IOException'. | 119 | |
First sentence should end with a period. | 121 | |
Expected @throws tag for 'IOException'. | 127 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 23 | |
First sentence should end with a period. | 27 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 21 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 23 | |
First sentence should end with a period. | 27 |
Violation | Message | Line |
---|---|---|
Parameter msgList should be final. | 47 | |
Method 'setClass' is not designed for extension - needs to be abstract, final or empty. | 51 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 74 | |
Parameter context should be final. | 86 | |
Parameter service should be final. | 87 | |
Parameter graphState should be final. | 88 | |
Expected @throws tag for 'IOException'. | 89 | |
Expected @throws tag for 'InterruptedException'. | 89 | |
Method 'createJobToken' is not designed for extension - needs to be abstract, final or empty. | 98 | |
Expected @throws tag for 'IOException'. | 102 | |
Method 'getRPCServer' is not designed for extension - needs to be abstract, final or empty. | 127 | |
Parameter myAddress should be final. | 129 | |
Parameter numHandlers should be final. | 129 | |
Parameter jobId should be final. | 129 | |
Parameter jt should be final. | 135 | |
Expected @throws tag for 'IOException'. | 135 | |
Method 'getRPCProxy' is not designed for extension - needs to be abstract, final or empty. | 166 | |
Parameter jobId should be final. | 169 | |
Parameter jt should be final. | 173 | |
Expected @throws tag for 'IOException'. | 175 | |
Expected @throws tag for 'InterruptedException'. | 175 | |
'+' should be on a new line. | 185 | |
Method 'getServerData' is not designed for extension - needs to be abstract, final or empty. | 206 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 45 | |
First sentence should end with a period. | 47 | |
First sentence should end with a period. | 50 | |
First sentence should end with a period. | 54 | |
Parameter conf should be final. | 59 | |
Method 'addMessage' is not designed for extension - needs to be abstract, final or empty. | 75 | |
Parameter partitionId should be final. | 75 | |
Parameter destVertexId should be final. | 75 | |
Parameter message should be final. | 75 | |
Method 'removePartitionMessages' is not designed for extension - needs to be abstract, final or empty. | 118 | |
Parameter partitionId should be final. | 118 | |
Method 'removeAllPartitionMessages' is not designed for extension - needs to be abstract, final or empty. | 129 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 42 | |
First sentence should end with a period. | 45 | |
Parameter partitionId should be final. | 58 | |
Parameter destVertexId should be final. | 58 | |
Parameter partitionId should be final. | 79 | |
Method 'addEdgeMutation' is not designed for extension - needs to be abstract, final or empty. | 98 | |
Parameter partitionId should be final. | 99 | |
Parameter destVertexId should be final. | 99 | |
Parameter edge should be final. | 99 | |
Method 'removeEdgeMutation' is not designed for extension - needs to be abstract, final or empty. | 119 | |
Parameter partitionId should be final. | 120 | |
Parameter vertexIndex should be final. | 120 | |
Parameter destinationVertexIndex should be final. | 120 | |
Method 'addVertexMutation' is not designed for extension - needs to be abstract, final or empty. | 139 | |
Parameter partitionId should be final. | 140 | |
Parameter vertex should be final. | 140 | |
Method 'removeVertexMutation' is not designed for extension - needs to be abstract, final or empty. | 159 | |
Parameter partitionId should be final. | 160 | |
Parameter destVertexId should be final. | 160 | |
Method 'removePartitionMutations' is not designed for extension - needs to be abstract, final or empty. | 178 | |
Parameter partitionId should be final. | 179 | |
Method 'removeAllPartitionMutations' is not designed for extension - needs to be abstract, final or empty. | 191 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 35 | |
First sentence should end with a period. | 48 | |
First sentence should end with a period. | 51 | |
First sentence should end with a period. | 56 | |
Parameter configuration should be final. | 74 | |
Parameter messageStoreFactory should be final. | 75 | |
'messageStoreFactory' hides a field. | 76 | |
Method 'getPartitionStore' is not designed for extension - needs to be abstract, final or empty. | 94 | |
First sentence should end with a period. | 98 | |
Method 'getIncomingMessageStore' is not designed for extension - needs to be abstract, final or empty. | 104 | |
First sentence should end with a period. | 108 | |
Method 'getCurrentMessageStore' is not designed for extension - needs to be abstract, final or empty. | 114 | |
First sentence should end with a period. | 118 | |
Method 'prepareSuperstep' is not designed for extension - needs to be abstract, final or empty. | 119 | |
First sentence should end with a period. | 132 | |
Method 'getVertexMutations' is not designed for extension - needs to be abstract, final or empty. | 137 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 41 | |
First sentence should end with a period. | 43 | |
First sentence should end with a period. | 45 | |
Parameter vertexId should be final. | 59 | |
'vertexId' hides a field. | 59 | |
Parameter msgList should be final. | 59 | |
'msgList' hides a field. | 59 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 64 | |
Parameter input should be final. | 65 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 73 | |
Parameter output should be final. | 74 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 79 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 84 | |
Parameter conf should be final. | 85 | |
'conf' hides a field. | 85 | |
Method 'getVertexId' is not designed for extension - needs to be abstract, final or empty. | 94 | |
Method 'getMessageList' is not designed for extension - needs to be abstract, final or empty. | 103 |
Violation | Message | Line |
---|---|---|
Parameter vertexIdMessagesList should be final. | 50 | |
Method 'setClass' is not designed for extension - needs to be abstract, final or empty. | 54 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 44 | |
Method 'setClass' is not designed for extension - needs to be abstract, final or empty. | 49 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 31 | |
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 70 | |
Expected @throws tag for 'IOException'. | 77 | |
First sentence should end with a period. | 79 | |
Expected @throws tag for 'IOException'. | 87 | |
First sentence should end with a period. | 89 | |
Expected @throws tag for 'IOException'. | 95 | |
First sentence should end with a period. | 97 | |
Expected @throws tag for 'IOException'. | 103 | |
Expected @throws tag for 'IOException'. | 111 | |
Expected @throws tag for 'IOException'. | 125 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 24 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 51 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 28 | |
First sentence should end with a period. | 36 | |
Expected @throws tag for 'IOException'. | 42 | |
Expected @throws tag for 'IOException'. | 51 | |
Expected @throws tag for 'IOException'. | 59 | |
Expected @throws tag for 'IOException'. | 66 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 57 | |
First sentence should end with a period. | 59 | |
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 63 | |
First sentence should end with a period. | 65 | |
First sentence should end with a period. | 67 | |
First sentence should end with a period. | 70 | |
Parameter combiner should be final. | 78 | |
'combiner' hides a field. | 78 | |
Parameter config should be final. | 79 | |
'config' hides a field. | 79 | |
Parameter fileStoreFactory should be final. | 80 | |
'fileStoreFactory' hides a field. | 80 | |
Method 'addVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 91 | |
Parameter vertexId should be final. | 92 | |
Parameter messages should be final. | 93 | |
Method 'addMessages' is not designed for extension - needs to be abstract, final or empty. | 117 | |
Parameter messages should be final. | 118 | |
Method 'getVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 124 | |
Parameter vertexId should be final. | 125 | |
Method 'getNumberOfMessages' is not designed for extension - needs to be abstract, final or empty. | 136 | |
Method 'hasMessagesForVertex' is not designed for extension - needs to be abstract, final or empty. | 141 | |
Parameter vertexId should be final. | 142 | |
Method 'getDestinationVertices' is not designed for extension - needs to be abstract, final or empty. | 146 | |
Method 'clearVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 151 | |
Parameter vertexId should be final. | 152 | |
Method 'clearAll' is not designed for extension - needs to be abstract, final or empty. | 156 | |
Method 'flush' is not designed for extension - needs to be abstract, final or empty. | 166 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 184 | |
Parameter out should be final. | 185 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 209 | |
Parameter in should be final. | 210 | |
First sentence should end with a period. | 245 | |
Parameter config should be final. | 257 | |
Parameter fileStoreFactory should be final. | 258 | |
First sentence should end with a period. | 262 | |
First sentence should end with a period. | 271 | |
First sentence should end with a period. | 273 | |
First sentence should end with a period. | 275 | |
Parameter config should be final. | 284 | |
'config' hides a field. | 284 | |
Parameter fileStoreFactory should be final. | 285 | |
'fileStoreFactory' hides a field. | 285 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 51 | |
First sentence should end with a period. | 53 | |
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 58 | |
Parameter service should be final. | 69 | |
'service' hides a field. | 69 | |
Parameter maxNumberOfMessagesInMemory should be final. | 70 | |
'maxNumberOfMessagesInMemory' hides a field. | 70 | |
Parameter fileStoreFactory should be final. | 71 | |
'fileStoreFactory' hides a field. | 72 | |
Method 'addVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 79 | |
Parameter vertexId should be final. | 80 | |
Parameter messages should be final. | 81 | |
Method 'addMessages' is not designed for extension - needs to be abstract, final or empty. | 86 | |
Parameter messages should be final. | 87 | |
Method 'addPartitionMessages' is not designed for extension - needs to be abstract, final or empty. | 95 | |
Parameter messages should be final. | 96 | |
Parameter partitionId should be final. | 97 | |
Method 'getVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 102 | |
Parameter vertexId should be final. | 103 | |
Method 'getNumberOfMessages' is not designed for extension - needs to be abstract, final or empty. | 111 | |
':' should be on a new line. | 114 | |
Method 'hasMessagesForVertex' is not designed for extension - needs to be abstract, final or empty. | 121 | |
Parameter vertexId should be final. | 122 | |
Method 'getDestinationVertices' is not designed for extension - needs to be abstract, final or empty. | 126 | |
':' should be on a new line. | 129 | |
Method 'getPartitionDestinationVertices' is not designed for extension - needs to be abstract, final or empty. | 136 | |
Parameter partitionId should be final. | 137 | |
Method 'clearVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 147 | |
Parameter vertexId should be final. | 148 | |
Method 'clearPartition' is not designed for extension - needs to be abstract, final or empty. | 154 | |
Parameter partitionId should be final. | 155 | |
Method 'clearAll' is not designed for extension - needs to be abstract, final or empty. | 163 | |
':' should be on a new line. | 165 | |
First sentence should end with a period. | 172 | |
Expected @throws tag for 'IOException'. | 177 | |
First sentence should end with a period. | 183 | |
':' should be on a new line. | 190 | |
First sentence should end with a period. | 197 | |
Expected @throws tag for 'IOException'. | 202 | |
':' should be on a new line. | 205 | |
First sentence should end with a period. | 218 | |
Parameter vertexId should be final. | 225 | |
Parameter partitionId should be final. | 239 | |
Avoid inline conditionals. | 248 | |
Method 'writePartition' is not designed for extension - needs to be abstract, final or empty. | 251 | |
Parameter out should be final. | 252 | |
Parameter partitionId should be final. | 253 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 262 | |
Parameter out should be final. | 263 | |
':' should be on a new line. | 265 | |
Method 'readFieldsForPartition' is not designed for extension - needs to be abstract, final or empty. | 272 | |
Parameter in should be final. | 273 | |
Parameter partitionId should be final. | 274 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 282 | |
Parameter in should be final. | 283 | |
First sentence should end with a period. | 294 | |
Parameter service should be final. | 310 | |
Parameter maxMessagesInMemory should be final. | 311 | |
Parameter fileStoreFactory should be final. | 312 | |
First sentence should end with a period. | 318 | |
First sentence should end with a period. | 329 | |
First sentence should end with a period. | 331 | |
First sentence should end with a period. | 333 | |
Parameter service should be final. | 343 | |
'service' hides a field. | 343 | |
Parameter maxMessagesInMemory should be final. | 344 | |
'maxMessagesInMemory' hides a field. | 344 | |
Parameter fileStoreFactory should be final. | 345 | |
'fileStoreFactory' hides a field. | 346 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 26 | |
Expected @throws tag for 'IOException'. | 39 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 35 | |
Expected @throws tag for 'IOException'. | 43 | |
First sentence should end with a period. | 45 | |
First sentence should end with a period. | 52 | |
First sentence should end with a period. | 60 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 30 | |
First sentence should end with a period. | 38 | |
Expected @throws tag for 'IOException'. | 46 | |
First sentence should end with a period. | 48 | |
Expected @throws tag for 'IOException'. | 62 | |
Expected @throws tag for 'IOException'. | 71 | |
First sentence should end with a period. | 73 | |
Expected @throws tag for 'IOException'. | 82 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 24 | |
First sentence should end with a period. | 33 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 63 | |
First sentence should end with a period. | 65 | |
First sentence should end with a period. | 67 | |
First sentence should end with a period. | 69 | |
First sentence should end with a period. | 71 | |
Parameter config should be final. | 82 | |
'config' hides a field. | 82 | |
Parameter bufferSize should be final. | 82 | |
'bufferSize' hides a field. | 82 | |
Parameter fileName should be final. | 83 | |
Method 'addMessages' is not designed for extension - needs to be abstract, final or empty. | 89 | |
Parameter messages should be final. | 90 | |
Parameter messages should be final. | 107 | |
Expected @throws tag for 'IOException'. | 108 | |
Method 'getVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 143 | |
Parameter vertexId should be final. | 144 | |
Expected @throws tag for 'IOException'. | 145 | |
Parameter vertexId should be final. | 162 | |
Method 'clearAll' is not designed for extension - needs to be abstract, final or empty. | 165 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 171 | |
Parameter out should be final. | 172 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 189 | |
Parameter in should be final. | 190 | |
'in' hides a field. | 190 | |
First sentence should end with a period. | 205 | |
Expected @throws tag for 'IOException'. | 210 | |
Expected @throws tag for 'IOException'. | 227 | |
Expected @throws tag for 'IOException'. | 244 | |
Expected @throws tag for 'IOException'. | 262 | |
Expected @throws tag for 'IOException'. | 279 | |
Expected @throws tag for 'IOException'. | 292 | |
First sentence should end with a period. | 300 | |
Parameter config should be final. | 310 | |
First sentence should end with a period. | 314 | |
First sentence should end with a period. | 323 | |
First sentence should end with a period. | 325 | |
First sentence should end with a period. | 327 | |
First sentence should end with a period. | 329 | |
Parameter config should be final. | 333 | |
'config' hides a field. | 333 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 42 | |
First sentence should end with a period. | 53 | |
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 60 | |
First sentence should end with a period. | 62 | |
Parameter service should be final. | 70 | |
'service' hides a field. | 70 | |
Parameter combiner should be final. | 71 | |
'combiner' hides a field. | 71 | |
Parameter config should be final. | 71 | |
'config' hides a field. | 71 | |
Method 'addVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Parameter vertexId should be final. | 79 | |
Parameter messages should be final. | 80 | |
Method 'addMessages' is not designed for extension - needs to be abstract, final or empty. | 102 | |
Parameter messages should be final. | 103 | |
Method 'addPartitionMessages' is not designed for extension - needs to be abstract, final or empty. | 109 | |
Parameter messages should be final. | 110 | |
Parameter partitionId should be final. | 111 | |
Method 'getVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 115 | |
Parameter vertexId should be final. | 116 | |
Avoid inline conditionals. | 119 | |
':' should be on a new line. | 119 | |
Method 'getNumberOfMessages' is not designed for extension - needs to be abstract, final or empty. | 123 | |
Method 'hasMessagesForVertex' is not designed for extension - needs to be abstract, final or empty. | 134 | |
Parameter vertexId should be final. | 135 | |
Avoid inline conditionals. | 138 | |
Method 'getDestinationVertices' is not designed for extension - needs to be abstract, final or empty. | 141 | |
Method 'getPartitionDestinationVertices' is not designed for extension - needs to be abstract, final or empty. | 150 | |
Parameter partitionId should be final. | 151 | |
Avoid inline conditionals. | 153 | |
':' should be on a new line. | 153 | |
Method 'clearVertexMessages' is not designed for extension - needs to be abstract, final or empty. | 157 | |
Parameter vertexId should be final. | 158 | |
Method 'clearPartition' is not designed for extension - needs to be abstract, final or empty. | 166 | |
Parameter partitionId should be final. | 167 | |
Method 'clearAll' is not designed for extension - needs to be abstract, final or empty. | 171 | |
First sentence should end with a period. | 176 | |
Parameter vertexId should be final. | 182 | |
Method 'writePartition' is not designed for extension - needs to be abstract, final or empty. | 186 | |
Parameter out should be final. | 187 | |
Parameter partitionId should be final. | 188 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 203 | |
Parameter out should be final. | 204 | |
Method 'readFieldsForPartition' is not designed for extension - needs to be abstract, final or empty. | 212 | |
Parameter in should be final. | 213 | |
Parameter partitionId should be final. | 214 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 234 | |
Parameter in should be final. | 235 | |
First sentence should end with a period. | 244 | |
Parameter service should be final. | 258 | |
Parameter config should be final. | 258 | |
First sentence should end with a period. | 262 | |
First sentence should end with a period. | 273 | |
First sentence should end with a period. | 275 | |
First sentence should end with a period. | 277 | |
Parameter service should be final. | 284 | |
'service' hides a field. | 284 | |
Parameter config should be final. | 285 | |
'config' hides a field. | 285 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 34 | |
First sentence should end with a period. | 36 | |
First sentence should end with a period. | 39 | |
First sentence should end with a period. | 41 | |
First sentence should end with a period. | 43 | |
First sentence should end with a period. | 45 | |
First sentence should end with a period. | 47 | |
Method 'handleUpstream' is not designed for extension - needs to be abstract, final or empty. | 51 | |
Parameter ctx should be final. | 52 | |
Parameter e should be final. | 52 | |
'&&' should be on a new line. | 54 | |
Method 'handleDownstream' is not designed for extension - needs to be abstract, final or empty. | 64 | |
Parameter ctx should be final. | 65 | |
Parameter e should be final. | 65 | |
'&&' should be on a new line. | 67 | |
First sentence should end with a period. | 94 | |
Method 'resetAll' is not designed for extension - needs to be abstract, final or empty. | 97 | |
Method 'getBytesSent' is not designed for extension - needs to be abstract, final or empty. | 102 | |
Missing a Javadoc comment. | 102 | |
Method 'getBytesReceived' is not designed for extension - needs to be abstract, final or empty. | 106 | |
Missing a Javadoc comment. | 106 | |
Method 'getMbytesPerSecSent' is not designed for extension - needs to be abstract, final or empty. | 113 | |
'1000f' is a magic number. | 114 | |
'/' should be on a new line. | 114 | |
Method 'getMbytesPerSecReceived' is not designed for extension - needs to be abstract, final or empty. | 121 | |
'1000f' is a magic number. | 122 | |
'/' should be on a new line. | 122 | |
Method 'getMetrics' is not designed for extension - needs to be abstract, final or empty. | 129 | |
Avoid inline conditionals. | 135 | |
Avoid inline conditionals. | 137 | |
'+' should be on a new line. | 138 | |
'+' should be on a new line. | 139 | |
'+' should be on a new line. | 140 | |
'+' should be on a new line. | 141 | |
'+' should be on a new line. | 142 | |
'+' should be on a new line. | 143 | |
'+' should be on a new line. | 144 | |
'+' should be on a new line. | 145 | |
'+' should be on a new line. | 146 | |
'+' should be on a new line. | 147 | |
'+' should be on a new line. | 148 | |
'1000f' is a magic number. | 149 | |
Method 'getMetricsWindow' is not designed for extension - needs to be abstract, final or empty. | 159 | |
Parameter minMsecsWindow should be final. | 159 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 26 | |
First sentence should end with a period. | 30 | |
First sentence should end with a period. | 32 | |
First sentence should end with a period. | 35 | |
Method 'addChannel' is not designed for extension - needs to be abstract, final or empty. | 40 | |
Parameter channel should be final. | 40 | |
First sentence should end with a period. | 44 | |
Method 'nextChannel' is not designed for extension - needs to be abstract, final or empty. | 49 | |
First sentence should end with a period. | 58 | |
Method 'removeLast' is not designed for extension - needs to be abstract, final or empty. | 63 | |
First sentence should end with a period. | 69 | |
First sentence should end with a period. | 79 | |
Method 'getChannels' is not designed for extension - needs to be abstract, final or empty. | 84 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 64 | |
First sentence should end with a period. | 66 | |
First sentence should end with a period. | 69 | |
First sentence should end with a period. | 71 | |
First sentence should end with a period. | 73 | |
First sentence should end with a period. | 75 | |
First sentence should end with a period. | 77 | |
First sentence should end with a period. | 79 | |
First sentence should end with a period. | 81 | |
First sentence should end with a period. | 92 | |
First sentence should end with a period. | 94 | |
First sentence should end with a period. | 96 | |
First sentence should end with a period. | 98 | |
First sentence should end with a period. | 100 | |
First sentence should end with a period. | 102 | |
First sentence should end with a period. | 104 | |
First sentence should end with a period. | 106 | |
First sentence should end with a period. | 108 | |
First sentence should end with a period. | 110 | |
First sentence should end with a period. | 112 | |
First sentence should end with a period. | 114 | |
First sentence should end with a period. | 116 | |
First sentence should end with a period. | 118 | |
First sentence should end with a period. | 121 | |
First sentence should end with a period. | 124 | |
Parameter context should be final. | 129 | |
'context' hides a field. | 129 | |
'+' should be on a new line. | 148 | |
First sentence should end with a period. | 215 | |
First sentence should end with a period. | 217 | |
Parameter future should be final. | 226 | |
'future' hides a field. | 226 | |
Parameter address should be final. | 226 | |
'address' hides a field. | 226 | |
First sentence should end with a period. | 232 | |
Method 'connectAllAddresses' is not designed for extension - needs to be abstract, final or empty. | 237 | |
Parameter addresses should be final. | 237 | |
'||' should be on a new line. | 242 | |
'+' should be on a new line. | 244 | |
'+' should be on a new line. | 248 | |
'+' should be on a new line. | 275 | |
'+' should be on a new line. | 276 | |
'+' should be on a new line. | 287 | |
'+' should be on a new line. | 306 | |
'+' should be on a new line. | 307 | |
'+' should be on a new line. | 308 | |
'+' should be on a new line. | 309 | |
Method 'stop' is not designed for extension - needs to be abstract, final or empty. | 325 | |
Parameter cf should be final. | 340 | |
'+' should be on a new line. | 344 | |
'+' should be on a new line. | 345 | |
First sentence should end with a period. | 358 | |
Parameter remoteServer should be final. | 364 | |
'+' should be on a new line. | 379 | |
'+' should be on a new line. | 380 | |
'+' should be on a new line. | 396 | |
'+' should be on a new line. | 397 | |
'5000' is a magic number. | 401 | |
'+' should be on a new line. | 406 | |
'+' should be on a new line. | 407 | |
First sentence should end with a period. | 411 | |
Method 'sendWritableRequest' is not designed for extension - needs to be abstract, final or empty. | 418 | |
Parameter destWorkerId should be final. | 418 | |
Parameter remoteServer should be final. | 419 | |
Parameter request should be final. | 420 | |
'+' should be on a new line. | 435 | |
'+' should be on a new line. | 436 | |
'&&' should be on a new line. | 442 | |
Method 'waitAllRequests' is not designed for extension - needs to be abstract, final or empty. | 453 | |
'+' should be on a new line. | 456 | |
Parameter maxOpenRequests should be final. | 469 | |
'+' should be on a new line. | 476 | |
'+' should be on a new line. | 477 | |
'+' should be on a new line. | 478 | |
'+' should be on a new line. | 479 | |
':' should be on a new line. | 483 | |
'+' should be on a new line. | 485 | |
':' should be on a new line. | 501 | |
'||' should be on a new line. | 507 | |
'||' should be on a new line. | 508 | |
'+' should be on a new line. | 510 | |
'+' should be on a new line. | 511 | |
'+' should be on a new line. | 512 | |
'+' should be on a new line. | 513 | |
'+' should be on a new line. | 514 | |
'+' should be on a new line. | 515 | |
'+' should be on a new line. | 516 | |
'+' should be on a new line. | 517 | |
'==' should be on a new line. | 530 | |
'+' should be on a new line. | 532 | |
Unused @throws tag for 'ArrayIndexOutOfBoundsException'. | 551 | |
Method 'getChannelConfig' is not designed for extension - needs to be abstract, final or empty. | 555 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 33 | |
First sentence should end with a period. | 37 | |
First sentence should end with a period. | 39 | |
First sentence should end with a period. | 42 | |
Parameter context should be final. | 47 | |
Method 'fixWorkerAddresses' is not designed for extension - needs to be abstract, final or empty. | 52 | |
Parameter workers should be final. | 53 | |
'workers' hides a field. | 53 | |
Method 'flush' is not designed for extension - needs to be abstract, final or empty. | 64 | |
Method 'closeConnections' is not designed for extension - needs to be abstract, final or empty. | 69 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 33 | |
First sentence should end with a period. | 35 | |
First sentence should end with a period. | 38 | |
Parameter context should be final. | 43 | |
Method 'fixWorkerAddresses' is not designed for extension - needs to be abstract, final or empty. | 48 | |
Parameter workers should be final. | 49 | |
Method 'flush' is not designed for extension - needs to be abstract, final or empty. | 53 | |
Method 'closeConnections' is not designed for extension - needs to be abstract, final or empty. | 58 | |
Method 'getMyAddress' is not designed for extension - needs to be abstract, final or empty. | 63 | |
Method 'close' is not designed for extension - needs to be abstract, final or empty. | 68 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 31 | |
First sentence should end with a period. | 34 | |
Parameter conf should be final. | 39 | |
Method 'getMyAddress' is not designed for extension - needs to be abstract, final or empty. | 45 | |
Method 'close' is not designed for extension - needs to be abstract, final or empty. | 50 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 47 | |
First sentence should end with a period. | 51 | |
First sentence should end with a period. | 53 | |
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 57 | |
First sentence should end with a period. | 59 | |
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 63 | |
First sentence should end with a period. | 65 | |
First sentence should end with a period. | 67 | |
First sentence should end with a period. | 69 | |
First sentence should end with a period. | 71 | |
First sentence should end with a period. | 73 | |
First sentence should end with a period. | 75 | |
First sentence should end with a period. | 77 | |
First sentence should end with a period. | 79 | |
First sentence should end with a period. | 81 | |
First sentence should end with a period. | 83 | |
First sentence should end with a period. | 86 | |
Parameter conf should be final. | 92 | |
'conf' hides a field. | 92 | |
Parameter requestServerHandlerFactory should be final. | 93 | |
'requestServerHandlerFactory' hides a field. | 93 | |
First sentence should end with a period. | 128 | |
Method 'start' is not designed for extension - needs to be abstract, final or empty. | 131 | |
'1024' is a magic number. | 144 | |
'1024' is a magic number. | 144 | |
'1024' is a magic number. | 144 | |
'4' is a magic number. | 144 | |
'4' is a magic number. | 144 | |
'10' is a magic number. | 156 | |
'+' should be on a new line. | 158 | |
'+' should be on a new line. | 176 | |
'+' should be on a new line. | 177 | |
'+' should be on a new line. | 191 | |
'+' should be on a new line. | 199 | |
'+' should be on a new line. | 204 | |
'+' should be on a new line. | 205 | |
'+' should be on a new line. | 206 | |
'+' should be on a new line. | 207 | |
'+' should be on a new line. | 208 | |
Method 'stop' is not designed for extension - needs to be abstract, final or empty. | 216 | |
Method 'getMyAddress' is not designed for extension - needs to be abstract, final or empty. | 227 | |
Missing a Javadoc comment. | 227 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 70 | |
First sentence should end with a period. | 73 | |
First sentence should end with a period. | 75 | |
First sentence should end with a period. | 77 | |
First sentence should end with a period. | 84 | |
First sentence should end with a period. | 88 | |
First sentence should end with a period. | 92 | |
First sentence should end with a period. | 94 | |
First sentence should end with a period. | 96 | |
First sentence should end with a period. | 98 | |
First sentence should end with a period. | 100 | |
Parameter context should be final. | 110 | |
Parameter service should be final. | 111 | |
'service' hides a field. | 111 | |
Parameter serverData should be final. | 112 | |
'serverData' hides a field. | 112 | |
Method 'fixPartitionIdToSocketAddrMap' is not designed for extension - needs to be abstract, final or empty. | 128 | |
'&&' should be on a new line. | 138 | |
'||' should be on a new line. | 140 | |
'!=' should be on a new line. | 141 | |
'+' should be on a new line. | 144 | |
'+' should be on a new line. | 145 | |
'+' should be on a new line. | 146 | |
'!=' should be on a new line. | 154 | |
Parameter workerInfo should be final. | 175 | |
Parameter partitionId should be final. | 176 | |
'&&' should be on a new line. | 182 | |
'+' should be on a new line. | 186 | |
'+' should be on a new line. | 187 | |
'5000' is a magic number. | 190 | |
'+' should be on a new line. | 196 | |
First sentence should end with a period. | 205 | |
Parameter workerInfo should be final. | 212 | |
Parameter remoteServerAddress should be final. | 213 | |
Parameter writableRequest should be final. | 214 | |
'==' should be on a new line. | 216 | |
Method 'sendMessageRequest' is not designed for extension - needs to be abstract, final or empty. | 225 | |
Parameter destVertexId should be final. | 226 | |
Parameter message should be final. | 226 | |
'+' should be on a new line. | 231 | |
Method 'sendPartitionRequest' is not designed for extension - needs to be abstract, final or empty. | 254 | |
Parameter workerInfo should be final. | 255 | |
Parameter partition should be final. | 256 | |
'+' should be on a new line. | 260 | |
':' should be on a new line. | 275 | |
Parameter partitionId should be final. | 309 | |
Parameter partitionOwner should be final. | 309 | |
Parameter partitionMutationCount should be final. | 310 | |
Method 'addEdgeRequest' is not designed for extension - needs to be abstract, final or empty. | 325 | |
Parameter vertexIndex should be final. | 326 | |
Parameter edge should be final. | 326 | |
'+' should be on a new line. | 332 | |
Method 'removeEdgeRequest' is not designed for extension - needs to be abstract, final or empty. | 344 | |
Parameter vertexIndex should be final. | 345 | |
Parameter destinationVertexIndex should be final. | 346 | |
'+' should be on a new line. | 351 | |
Method 'addVertexRequest' is not designed for extension - needs to be abstract, final or empty. | 364 | |
Parameter vertex should be final. | 365 | |
'+' should be on a new line. | 370 | |
Method 'removeVertexRequest' is not designed for extension - needs to be abstract, final or empty. | 382 | |
Parameter vertexIndex should be final. | 383 | |
'+' should be on a new line. | 388 | |
Method 'flush' is not designed for extension - needs to be abstract, final or empty. | 400 | |
':' should be on a new line. | 405 | |
':' should be on a new line. | 423 | |
Method 'resetMessageCount' is not designed for extension - needs to be abstract, final or empty. | 441 | |
Method 'closeConnections' is not designed for extension - needs to be abstract, final or empty. | 448 | |
Method 'setup' is not designed for extension - needs to be abstract, final or empty. | 453 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 48 | |
First sentence should end with a period. | 50 | |
Parameter context should be final. | 59 | |
Parameter service should be final. | 60 | |
Method 'fixPartitionIdToSocketAddrMap' is not designed for extension - needs to be abstract, final or empty. | 67 | |
Method 'sendMessageRequest' is not designed for extension - needs to be abstract, final or empty. | 72 | |
Parameter destVertexId should be final. | 73 | |
Parameter message should be final. | 73 | |
Method 'sendPartitionRequest' is not designed for extension - needs to be abstract, final or empty. | 77 | |
Parameter workerInfo should be final. | 78 | |
Parameter partition should be final. | 79 | |
Method 'addEdgeRequest' is not designed for extension - needs to be abstract, final or empty. | 83 | |
Parameter vertexIndex should be final. | 84 | |
Parameter edge should be final. | 84 | |
Method 'removeEdgeRequest' is not designed for extension - needs to be abstract, final or empty. | 89 | |
Parameter vertexIndex should be final. | 90 | |
Parameter destinationVertexIndex should be final. | 91 | |
Method 'addVertexRequest' is not designed for extension - needs to be abstract, final or empty. | 95 | |
Parameter vertex should be final. | 96 | |
Method 'removeVertexRequest' is not designed for extension - needs to be abstract, final or empty. | 100 | |
Parameter vertexIndex should be final. | 101 | |
Method 'flush' is not designed for extension - needs to be abstract, final or empty. | 105 | |
Method 'resetMessageCount' is not designed for extension - needs to be abstract, final or empty. | 110 | |
Method 'closeConnections' is not designed for extension - needs to be abstract, final or empty. | 115 | |
Method 'setup' is not designed for extension - needs to be abstract, final or empty. | 120 | |
Method 'prepareSuperstep' is not designed for extension - needs to be abstract, final or empty. | 125 | |
Method 'getServerData' is not designed for extension - needs to be abstract, final or empty. | 130 | |
Method 'close' is not designed for extension - needs to be abstract, final or empty. | 135 | |
Method 'getPort' is not designed for extension - needs to be abstract, final or empty. | 141 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 64 | |
First sentence should end with a period. | 66 | |
First sentence should end with a period. | 68 | |
First sentence should end with a period. | 70 | |
Parameter conf should be final. | 79 | |
'conf' hides a field. | 79 | |
Parameter service should be final. | 80 | |
'service' hides a field. | 80 | |
Method 'getPort' is not designed for extension - needs to be abstract, final or empty. | 109 | |
Method 'prepareSuperstep' is not designed for extension - needs to be abstract, final or empty. | 114 | |
'+' should be on a new line. | 127 | |
'+' should be on a new line. | 138 | |
'+' should be on a new line. | 165 | |
'+' should be on a new line. | 166 | |
'+' should be on a new line. | 167 | |
'+' should be on a new line. | 168 | |
'+' should be on a new line. | 169 | |
'+' should be on a new line. | 177 | |
'+' should be on a new line. | 178 | |
'+' should be on a new line. | 189 | |
'+' should be on a new line. | 190 | |
Method 'getServerData' is not designed for extension - needs to be abstract, final or empty. | 195 | |
Method 'close' is not designed for extension - needs to be abstract, final or empty. | 200 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 25 | |
First sentence should end with a period. | 30 | |
Method 'getNextRequestId' is not designed for extension - needs to be abstract, final or empty. | 40 | |
Parameter address should be final. | 40 | |
'+' should be on a new line. | 45 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 29 | |
Parameter destinationWorkerId should be final. | 38 | |
'destinationWorkerId' hides a field. | 38 | |
Parameter requestId should be final. | 38 | |
'requestId' hides a field. | 38 | |
Method 'getDestinationWorkerId' is not designed for extension - needs to be abstract, final or empty. | 43 | |
Missing a Javadoc comment. | 43 | |
Method 'getRequestId' is not designed for extension - needs to be abstract, final or empty. | 47 | |
Missing a Javadoc comment. | 47 | |
Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 51 | |
'29' is a magic number. | 53 | |
'57' is a magic number. | 53 | |
Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 56 | |
Parameter other should be final. | 57 | |
'&&' should be on a new line. | 60 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 69 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 24 | |
First sentence should end with a period. | 27 | |
Parameter workerRequestReservedMap should be final. | 34 | |
Parameter conf should be final. | 34 | |
Method 'processRequest' is not designed for extension - needs to be abstract, final or empty. | 38 | |
Parameter request should be final. | 39 | |
First sentence should end with a period. | 43 | |
Method 'newHandler' is not designed for extension - needs to be abstract, final or empty. | 47 | |
Parameter workerRequestReservedMap should be final. | 49 | |
Parameter conf should be final. | 50 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 37 | |
First sentence should end with a period. | 40 | |
First sentence should end with a period. | 42 | |
Parameter conf should be final. | 51 | |
'conf' hides a field. | 51 | |
Parameter byteCounter should be final. | 51 | |
'byteCounter' hides a field. | 51 | |
Method 'decode' is not designed for extension - needs to be abstract, final or empty. | 56 | |
Parameter ctx should be final. | 57 | |
Parameter channel should be final. | 58 | |
Parameter msg should be final. | 58 | |
'30000' is a magic number. | 64 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 34 | |
First sentence should end with a period. | 36 | |
'4' is a magic number. | 37 | |
Method 'encode' is not designed for extension - needs to be abstract, final or empty. | 39 | |
Parameter ctx should be final. | 40 | |
Parameter channel should be final. | 41 | |
Parameter msg should be final. | 41 | |
'10' is a magic number. | 50 | |
'4' is a magic number. | 63 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 31 | |
First sentence should end with a period. | 33 | |
First sentence should end with a period. | 35 | |
First sentence should end with a period. | 37 | |
Parameter destinationAddress should be final. | 46 | |
'destinationAddress' hides a field. | 46 | |
Parameter request should be final. | 47 | |
'request' hides a field. | 47 | |
Method 'getDestinationAddress' is not designed for extension - needs to be abstract, final or empty. | 53 | |
Missing a Javadoc comment. | 53 | |
Method 'getStartedMsecs' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Missing a Javadoc comment. | 57 | |
Method 'getElapsedMsecs' is not designed for extension - needs to be abstract, final or empty. | 66 | |
Method 'getRequest' is not designed for extension - needs to be abstract, final or empty. | 70 | |
Missing a Javadoc comment. | 70 | |
Method 'setWriteFuture' is not designed for extension - needs to be abstract, final or empty. | 74 | |
Missing a Javadoc comment. | 74 | |
Parameter writeFuture should be final. | 74 | |
'writeFuture' hides a field. | 74 | |
Method 'getWriteFuture' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Missing a Javadoc comment. | 78 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 82 | |
'+' should be on a new line. | 84 | |
'+' should be on a new line. | 85 | |
'+' should be on a new line. | 86 | |
Avoid inline conditionals. | 87 | |
':' should be on a new line. | 87 | |
'+' should be on a new line. | 88 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 40 | |
First sentence should end with a period. | 42 | |
Name 'ALREADY_CLOSED_FIRST_REQUEST' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 46 | |
First sentence should end with a period. | 47 | |
First sentence should end with a period. | 49 | |
First sentence should end with a period. | 51 | |
First sentence should end with a period. | 54 | |
Parameter workerRequestReservedMap should be final. | 61 | |
'workerRequestReservedMap' hides a field. | 61 | |
Parameter conf should be final. | 62 | |
Method 'messageReceived' is not designed for extension - needs to be abstract, final or empty. | 70 | |
Parameter ctx should be final. | 72 | |
Parameter e should be final. | 72 | |
'+' should be on a new line. | 81 | |
'+' should be on a new line. | 82 | |
'+' should be on a new line. | 97 | |
'+' should be on a new line. | 98 | |
'+' should be on a new line. | 99 | |
'+' should be on a new line. | 100 | |
First sentence should end with a period. | 112 | |
Method 'channelConnected' is not designed for extension - needs to be abstract, final or empty. | 119 | |
Parameter ctx should be final. | 120 | |
Parameter e should be final. | 121 | |
'+' should be on a new line. | 123 | |
Method 'channelClosed' is not designed for extension - needs to be abstract, final or empty. | 128 | |
Parameter ctx should be final. | 129 | |
Parameter e should be final. | 130 | |
'+' should be on a new line. | 132 | |
'+' should be on a new line. | 133 | |
Method 'exceptionCaught' is not designed for extension - needs to be abstract, final or empty. | 138 | |
Parameter ctx should be final. | 139 | |
Parameter e should be final. | 139 | |
'+' should be on a new line. | 140 | |
First sentence should end with a period. | 144 | |
First sentence should end with a period. | 148 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 39 | |
Name 'ALREADY_DROPPED_FIRST_RESPONSE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 43 | |
First sentence should end with a period. | 44 | |
First sentence should end with a period. | 46 | |
Parameter workerIdOutstandingRequestMap should be final. | 58 | |
'workerIdOutstandingRequestMap' hides a field. | 59 | |
Parameter conf should be final. | 60 | |
Method 'messageReceived' is not designed for extension - needs to be abstract, final or empty. | 67 | |
Parameter ctx should be final. | 69 | |
Parameter event should be final. | 69 | |
'+' should be on a new line. | 71 | |
'+' should be on a new line. | 92 | |
'+' should be on a new line. | 111 | |
'+' should be on a new line. | 115 | |
'+' should be on a new line. | 116 | |
'+' should be on a new line. | 117 | |
Method 'channelClosed' is not designed for extension - needs to be abstract, final or empty. | 128 | |
Parameter ctx should be final. | 129 | |
Parameter e should be final. | 130 | |
'+' should be on a new line. | 132 | |
Method 'exceptionCaught' is not designed for extension - needs to be abstract, final or empty. | 137 | |
Parameter ctx should be final. | 138 | |
Parameter e should be final. | 138 | |
'+' should be on a new line. | 139 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 29 | |
First sentence should end with a period. | 33 | |
First sentence should end with a period. | 37 | |
Parameter conf should be final. | 42 | |
Method 'reserveRequest' is not designed for extension - needs to be abstract, final or empty. | 56 | |
Parameter workerId should be final. | 56 | |
Parameter requestId should be final. | 56 | |
Parameter workerId should be final. | 67 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 41 | |
Parameter serverData should be final. | 48 | |
'serverData' hides a field. | 48 | |
Parameter workerRequestReservedMap should be final. | 49 | |
Parameter conf should be final. | 50 | |
Method 'processRequest' is not designed for extension - needs to be abstract, final or empty. | 55 | |
Parameter request should be final. | 56 | |
First sentence should end with a period. | 60 | |
First sentence should end with a period. | 64 | |
First sentence should end with a period. | 67 | |
Parameter serverData should be final. | 72 | |
'serverData' hides a field. | 72 | |
Method 'newHandler' is not designed for extension - needs to be abstract, final or empty. | 76 | |
Parameter workerRequestReservedMap should be final. | 78 | |
Parameter conf should be final. | 79 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 21 | |
First sentence should end with a period. | 25 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 21 | |
First sentence should end with a period. | 25 | |
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 29 | |
First sentence should end with a period. | 35 | |
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 41 | |
Parameter requestClass should be final. | 46 | |
'requestClass' hides a field. | 46 | |
First sentence should end with a period. | 50 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 41 | |
Parameter partitionId should be final. | 50 | |
Parameter vertexIdMessages should be final. | 51 | |
Method 'getType' is not designed for extension - needs to be abstract, final or empty. | 55 | |
Method 'doRequest' is not designed for extension - needs to be abstract, final or empty. | 60 | |
Parameter serverData should be final. | 61 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 50 | |
First sentence should end with a period. | 53 | |
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 58 | |
Parameter partitionId should be final. | 69 | |
'partitionId' hides a field. | 69 | |
Parameter vertexIdMessages should be final. | 70 | |
'vertexIdMessages' hides a field. | 70 | |
Method 'readFieldsRequest' is not designed for extension - needs to be abstract, final or empty. | 75 | |
Parameter input should be final. | 76 | |
Method 'writeRequest' is not designed for extension - needs to be abstract, final or empty. | 97 | |
Parameter output should be final. | 98 | |
Method 'getType' is not designed for extension - needs to be abstract, final or empty. | 110 | |
Method 'doRequest' is not designed for extension - needs to be abstract, final or empty. | 115 | |
Parameter serverData should be final. | 116 | |
First sentence should end with a period. | 125 | |
Method 'getPartitionId' is not designed for extension - needs to be abstract, final or empty. | 130 | |
First sentence should end with a period. | 134 | |
Method 'getVertexIdMessages' is not designed for extension - needs to be abstract, final or empty. | 139 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 49 | |
First sentence should end with a period. | 52 | |
First sentence should end with a period. | 54 | |
First sentence should end with a period. | 57 | |
Parameter partitionId should be final. | 69 | |
'partitionId' hides a field. | 69 | |
Parameter vertexIdMutations should be final. | 70 | |
'vertexIdMutations' hides a field. | 70 | |
Method 'readFieldsRequest' is not designed for extension - needs to be abstract, final or empty. | 75 | |
Parameter input should be final. | 76 | |
Method 'writeRequest' is not designed for extension - needs to be abstract, final or empty. | 94 | |
Parameter output should be final. | 95 | |
':' should be on a new line. | 98 | |
Method 'getType' is not designed for extension - needs to be abstract, final or empty. | 105 | |
Method 'doRequest' is not designed for extension - needs to be abstract, final or empty. | 110 | |
Parameter serverData should be final. | 111 | |
':' should be on a new line. | 114 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 47 | |
First sentence should end with a period. | 50 | |
First sentence should end with a period. | 52 | |
First sentence should end with a period. | 55 | |
Parameter partitionId should be final. | 66 | |
'partitionId' hides a field. | 66 | |
Parameter vertices should be final. | 67 | |
'vertices' hides a field. | 67 | |
Method 'readFieldsRequest' is not designed for extension - needs to be abstract, final or empty. | 72 | |
Parameter input should be final. | 73 | |
Method 'writeRequest' is not designed for extension - needs to be abstract, final or empty. | 84 | |
Parameter output should be final. | 85 | |
Method 'getType' is not designed for extension - needs to be abstract, final or empty. | 93 | |
Method 'doRequest' is not designed for extension - needs to be abstract, final or empty. | 98 | |
Parameter serverData should be final. | 99 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 25 | |
First sentence should end with a period. | 35 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 28 | |
First sentence should end with a period. | 32 | |
First sentence should end with a period. | 34 | |
First sentence should end with a period. | 36 | |
Method 'getClientId' is not designed for extension - needs to be abstract, final or empty. | 39 | |
Missing a Javadoc comment. | 39 | |
Method 'setClientId' is not designed for extension - needs to be abstract, final or empty. | 43 | |
Missing a Javadoc comment. | 43 | |
Parameter clientId should be final. | 43 | |
'clientId' hides a field. | 43 | |
Method 'getRequestId' is not designed for extension - needs to be abstract, final or empty. | 47 | |
Missing a Javadoc comment. | 47 | |
Method 'setRequestId' is not designed for extension - needs to be abstract, final or empty. | 51 | |
Missing a Javadoc comment. | 51 | |
Parameter requestId should be final. | 51 | |
'requestId' hides a field. | 51 | |
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 62 | |
Expected @throws tag for 'IOException'. | 67 | |
First sentence should end with a period. | 69 | |
Expected @throws tag for 'IOException'. | 74 | |
Parameter conf should be final. | 82 | |
'conf' hides a field. | 82 | |
Parameter input should be final. | 87 | |
Parameter output should be final. | 94 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 35 | |
First sentence should end with a period. | 37 | |
First sentence should end with a period. | 39 | |
First sentence should end with a period. | 41 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 44 | |
Parameter messages should be final. | 45 | |
'10' is a magic number. | 65 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 73 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 93 | |
Parameter expected should be final. | 109 | |
Parameter actual should be final. | 109 | |
'+' should be on a new line. | 111 |
Violation | Message | Line |
---|---|---|
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 54 | |
Parameter messages should be final. | 55 | |
Expected @throws tag for 'IOException'. | 55 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 31 | |
First sentence should end with a period. | 43 | |
First sentence should end with a period. | 46 | |
First sentence should end with a period. | 51 | |
First sentence should end with a period. | 53 | |
Variable 'recordsRead' must be private and have accessor methods. | 54 | |
First sentence should end with a period. | 55 | |
Variable 'totalRecords' must be private and have accessor methods. | 56 | |
Variable 'inputSplit' must be private and have accessor methods. | 58 | |
Variable 'reverseIdOrder' must be private and have accessor methods. | 60 | |
First sentence should end with a period. | 61 | |
Variable 'configuration' must be private and have accessor methods. | 62 | |
Parameter inputSplit should be final. | 71 | |
'inputSplit' hides a field. | 71 | |
Parameter context should be final. | 72 | |
'100.0f' is a magic number. | 89 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 25 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 41 | |
Parameter messages should be final. | 42 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 29 | |
Method 'combine' is not designed for extension - needs to be abstract, final or empty. | 34 | |
Parameter target should be final. | 36 | |
Parameter messages should be final. | 37 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 28 | |
Method 'combine' is not designed for extension - needs to be abstract, final or empty. | 33 | |
Parameter target should be final. | 34 | |
Parameter messages should be final. | 35 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 38 | |
Name 'FILENAME' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 39 | |
First sentence should end with a period. | 40 | |
Missing a Javadoc comment. | 43 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 47 | |
Parameter context should be final. | 49 | |
Parameter applicationAttempt should be final. | 49 | |
Method 'writeAggregator' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Parameter aggregatorMap should be final. | 59 | |
Parameter superstep should be final. | 60 | |
Method 'close' is not designed for extension - needs to be abstract, final or empty. | 67 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 52 | |
First sentence should end with a period. | 54 | |
First sentence should end with a period. | 56 | |
First sentence should end with a period. | 62 | |
First sentence should end with a period. | 65 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 68 | |
Parameter messages should be final. | 69 | |
'&&' should be on a new line. | 76 | |
'&&' should be on a new line. | 77 | |
'+' should be on a new line. | 79 | |
'+' should be on a new line. | 80 | |
'+' should be on a new line. | 81 | |
'+' should be on a new line. | 94 | |
'+' should be on a new line. | 100 | |
'+' should be on a new line. | 101 | |
'+' should be on a new line. | 106 | |
'+' should be on a new line. | 107 | |
'+' should be on a new line. | 110 | |
'+' should be on a new line. | 112 | |
'+' should be on a new line. | 113 | |
'+' should be on a new line. | 114 | |
'+' should be on a new line. | 115 | |
'+' should be on a new line. | 116 | |
First sentence should end with a period. | 128 | |
First sentence should end with a period. | 130 | |
Name 'FINAL_SUM' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 131 | |
First sentence should end with a period. | 132 | |
'6' is a magic number. | 133 | |
Missing a Javadoc comment. | 137 | |
Method 'preApplication' is not designed for extension - needs to be abstract, final or empty. | 141 | |
Method 'postApplication' is not designed for extension - needs to be abstract, final or empty. | 150 | |
Method 'getSupersteps' is not designed for extension - needs to be abstract, final or empty. | 164 | |
Missing a Javadoc comment. | 164 | |
Method 'getEnableFault' is not designed for extension - needs to be abstract, final or empty. | 168 | |
Missing a Javadoc comment. | 168 | |
Method 'run' is not designed for extension - needs to be abstract, final or empty. | 173 | |
Parameter args should be final. | 174 | |
'100.0f' is a magic number. | 222 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 247 | |
Parameter args should be final. | 261 | |
Expected @throws tag for 'Exception'. | 261 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 265 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 270 | |
Parameter conf should be final. | 271 | |
'conf' hides a field. | 271 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 32 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 33 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 35 | |
Parameter messages should be final. | 36 | |
'101' is a magic number. | 38 | |
'102' is a magic number. | 39 | |
'103' is a magic number. | 40 | |
'+' should be on a new line. | 52 | |
'306' is a magic number. | 55 | |
'3' is a magic number. | 59 | |
'+' should be on a new line. | 61 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 32 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 33 | |
Comment matches to-do format 'TODO:'. | 34 | |
First sentence should end with a period. | 34 | |
Name 'SUPERSTEP' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 35 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 37 | |
Parameter messages should be final. | 38 | |
'0.15f' is a magic number. | 45 | |
'0.85f' is a magic number. | 45 | |
'30' is a magic number. | 47 | |
'20' is a magic number. | 48 | |
'10L' is a magic number. | 49 | |
'2000' is a magic number. | 51 | |
'10' is a magic number. | 56 |
Violation | Message | Line |
---|---|---|
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 36 | |
Parameter messages should be final. | 37 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 24 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 33 | |
First sentence should end with a period. | 35 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 39 | |
Parameter messages should be final. | 40 | |
Avoid inline conditionals. | 41 | |
First sentence should end with a period. | 56 | |
Name 'FINAL_SUM' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 57 | |
Method 'setFinalSum' is not designed for extension - needs to be abstract, final or empty. | 76 | |
Missing a Javadoc comment. | 76 | |
Parameter sum should be final. | 76 | |
Missing a Javadoc comment. | 80 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 90 | |
'10' is a magic number. | 94 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 99 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 32 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 33 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 34 | |
Parameter messages should be final. | 35 | |
'101' is a magic number. | 37 | |
'102' is a magic number. | 38 | |
'103' is a magic number. | 39 | |
'+' should be on a new line. | 49 | |
'306' is a magic number. | 52 | |
'3' is a magic number. | 56 | |
'+' should be on a new line. | 57 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 37 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 38 | |
First sentence should end with a period. | 40 | |
'100' is a magic number. | 41 | |
Parameter range should be final. | 51 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 55 | |
Parameter messages should be final. | 56 | |
'3' is a magic number. | 70 | |
'+' should be on a new line. | 74 | |
'+' should be on a new line. | 75 | |
'+' should be on a new line. | 81 | |
'+' should be on a new line. | 82 | |
'3' is a magic number. | 87 | |
'4' is a magic number. | 96 | |
'5' is a magic number. | 98 | |
'+' should be on a new line. | 102 | |
'+' should be on a new line. | 103 | |
'+' should be on a new line. | 109 | |
'+' should be on a new line. | 110 | |
'3' is a magic number. | 115 | |
'6' is a magic number. | 118 | |
'3' is a magic number. | 121 | |
'7' is a magic number. | 124 | |
'+' should be on a new line. | 128 | |
'+' should be on a new line. | 129 | |
'8' is a magic number. | 132 | |
'+' should be on a new line. | 136 | |
'+' should be on a new line. | 137 | |
First sentence should end with a period. | 150 | |
First sentence should end with a period. | 152 | |
First sentence should end with a period. | 154 | |
First sentence should end with a period. | 156 | |
Method 'postSuperstep' is not designed for extension - needs to be abstract, final or empty. | 169 | |
'+' should be on a new line. | 176 | |
'+' should be on a new line. | 177 | |
Method 'getVertexCount' is not designed for extension - needs to be abstract, final or empty. | 183 | |
Missing a Javadoc comment. | 183 | |
Method 'getEdgeCount' is not designed for extension - needs to be abstract, final or empty. | 187 | |
Missing a Javadoc comment. | 187 | |
Method 'getOrigEdgeCount' is not designed for extension - needs to be abstract, final or empty. | 191 | |
Missing a Javadoc comment. | 191 | |
Method 'increaseEdgesRemoved' is not designed for extension - needs to be abstract, final or empty. | 198 |
Violation | Message | Line |
---|---|---|
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 36 | |
Parameter messages should be final. | 37 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 57 | |
First sentence should end with a period. | 60 | |
Name 'SUM_AGG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 61 | |
First sentence should end with a period. | 62 | |
Name 'MIN_AGG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 63 | |
First sentence should end with a period. | 64 | |
Name 'MAX_AGG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 65 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 67 | |
Parameter messages should be final. | 68 | |
'0.15f' is a magic number. | 75 | |
'0.85f' is a magic number. | 75 | |
'+' should be on a new line. | 80 | |
'+' should be on a new line. | 81 | |
First sentence should end with a period. | 99 | |
Name 'FINAL_MAX' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 100 | |
First sentence should end with a period. | 101 | |
Name 'FINAL_MIN' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 102 | |
First sentence should end with a period. | 103 | |
Name 'FINAL_SUM' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 104 | |
Missing a Javadoc comment. | 106 | |
Missing a Javadoc comment. | 110 | |
Missing a Javadoc comment. | 114 | |
Method 'postApplication' is not designed for extension - needs to be abstract, final or empty. | 123 | |
Method 'preSuperstep' is not designed for extension - needs to be abstract, final or empty. | 134 | |
'3' is a magic number. | 136 | |
'+' should be on a new line. | 137 | |
'+' should be on a new line. | 138 | |
'!=' should be on a new line. | 140 | |
'+' should be on a new line. | 142 | |
'+' should be on a new line. | 143 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 163 | |
First sentence should end with a period. | 172 | |
First sentence should end with a period. | 178 | |
Method 'nextVertex' is not designed for extension - needs to be abstract, final or empty. | 189 | |
Method 'getCurrentVertex' is not designed for extension - needs to be abstract, final or empty. | 194 | |
'10d' is a magic number. | 203 | |
'%' should be on a new line. | 205 | |
'100f' is a magic number. | 207 | |
'+' should be on a new line. | 213 | |
'+' should be on a new line. | 214 | |
First sentence should end with a period. | 221 | |
Method 'createVertexReader' is not designed for extension - needs to be abstract, final or empty. | 227 | |
Parameter split should be final. | 229 | |
Parameter context should be final. | 230 | |
First sentence should end with a period. | 236 | |
Parameter lineRecordWriter should be final. | 247 | |
Method 'writeVertex' is not designed for extension - needs to be abstract, final or empty. | 251 | |
Parameter vertex should be final. | 253 | |
First sentence should end with a period. | 261 | |
Method 'createVertexWriter' is not designed for extension - needs to be abstract, final or empty. | 266 | |
Parameter context should be final. | 268 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 40 | |
First sentence should end with a period. | 42 | |
'==' should be on a new line. | 52 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Parameter messages should be final. | 58 | |
Avoid inline conditionals. | 62 | |
'+' should be on a new line. | 67 | |
'+' should be on a new line. | 75 |
Violation | Message | Line |
---|---|---|
Method 'combine' is not designed for extension - needs to be abstract, final or empty. | 36 | |
Parameter vertexIndex should be final. | 37 | |
Parameter messages should be final. | 38 |
Violation | Message | Line |
---|---|---|
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 49 | |
Parameter messages should be final. | 50 | |
'3' is a magic number. | 51 | |
First sentence should end with a period. | 56 | |
First sentence should end with a period. | 62 | |
Method 'nextVertex' is not designed for extension - needs to be abstract, final or empty. | 72 | |
Method 'getCurrentVertex' is not designed for extension - needs to be abstract, final or empty. | 77 | |
'?' should be on a new line. | 84 | |
Avoid inline conditionals. | 84 | |
'-' should be on a new line. | 85 | |
':' should be on a new line. | 86 | |
'10' is a magic number. | 90 | |
'%' should be on a new line. | 93 | |
'100f' is a magic number. | 95 | |
'+' should be on a new line. | 101 | |
'+' should be on a new line. | 102 | |
'+' should be on a new line. | 103 | |
First sentence should end with a period. | 110 | |
Method 'createVertexReader' is not designed for extension - needs to be abstract, final or empty. | 116 | |
Parameter split should be final. | 118 | |
Parameter context should be final. | 118 | |
First sentence should end with a period. | 124 | |
Parameter lineRecordWriter should be final. | 135 | |
Method 'writeVertex' is not designed for extension - needs to be abstract, final or empty. | 139 | |
Parameter vertex should be final. | 140 | |
First sentence should end with a period. | 148 | |
Method 'createVertexWriter' is not designed for extension - needs to be abstract, final or empty. | 153 | |
Parameter context should be final. | 155 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 67 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 71 | |
Parameter messages should be final. | 72 | |
'?' should be on a new line. | 80 | |
Avoid inline conditionals. | 80 | |
First sentence should end with a period. | 105 | |
First sentence should end with a period. | 107 | |
First sentence should end with a period. | 110 | |
First sentence should end with a period. | 113 | |
Parameter k should be final. | 117 | |
Parameter v should be final. | 117 | |
First sentence should end with a period. | 121 | |
Method 'getKey' is not designed for extension - needs to be abstract, final or empty. | 123 | |
First sentence should end with a period. | 124 | |
Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 126 | |
First sentence should end with a period. | 127 | |
Method 'compareTo' is not designed for extension - needs to be abstract, final or empty. | 130 | |
Parameter other should be final. | 131 | |
First sentence should end with a period. | 136 | |
First sentence should end with a period. | 140 | |
First sentence should end with a period. | 144 | |
Method 'setClass' is not designed for extension - needs to be abstract, final or empty. | 145 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 50 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 53 | |
Parameter messages should be final. | 54 | |
'+' should be on a new line. | 61 | |
First sentence should end with a period. | 73 | |
Method 'preApplication' is not designed for extension - needs to be abstract, final or empty. | 78 | |
'+' should be on a new line. | 90 | |
'+' should be on a new line. | 96 | |
'+' should be on a new line. | 100 | |
'+' should be on a new line. | 103 | |
Method 'postApplication' is not designed for extension - needs to be abstract, final or empty. | 114 | |
Method 'emit' is not designed for extension - needs to be abstract, final or empty. | 139 | |
Parameter s should be final. | 139 | |
Method 'run' is not designed for extension - needs to be abstract, final or empty. | 148 | |
Parameter args should be final. | 149 | |
'100.0f' is a magic number. | 163 | |
Parameter args should be final. | 177 | |
Expected @throws tag for 'Exception'. | 177 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 46 | |
First sentence should end with a period. | 48 | |
First sentence should end with a period. | 50 | |
Parameter superstep should be final. | 65 | |
'superstep' hides a field. | 65 | |
Parameter sourceVertexId should be final. | 65 | |
'sourceVertexId' hides a field. | 65 | |
Parameter value should be final. | 65 | |
'value' hides a field. | 65 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 71 | |
Parameter input should be final. | 72 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Parameter output should be final. | 79 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 85 | |
'+' should be on a new line. | 87 | |
First sentence should end with a period. | 98 | |
First sentence should end with a period. | 101 | |
Name 'FINAL_SUM' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 102 | |
First sentence should end with a period. | 103 | |
Name 'SUPERSTEPS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 104 | |
'6' is a magic number. | 104 | |
First sentence should end with a period. | 105 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 106 | |
Missing a Javadoc comment. | 108 | |
Method 'preApplication' is not designed for extension - needs to be abstract, final or empty. | 117 | |
Method 'postApplication' is not designed for extension - needs to be abstract, final or empty. | 124 | |
Method 'compute' is not designed for extension - needs to be abstract, final or empty. | 139 | |
Parameter messages should be final. | 140 | |
'+' should be on a new line. | 151 | |
'+' should be on a new line. | 152 | |
'+' should be on a new line. | 159 | |
'+' should be on a new line. | 160 | |
'+' should be on a new line. | 161 | |
'+' should be on a new line. | 166 | |
'+' should be on a new line. | 167 | |
'&&' should be on a new line. | 170 | |
'+' should be on a new line. | 173 | |
'+' should be on a new line. | 174 | |
'+' should be on a new line. | 181 | |
'+' should be on a new line. | 182 | |
'+' should be on a new line. | 189 | |
'+' should be on a new line. | 190 | |
'+' should be on a new line. | 191 | |
'+' should be on a new line. | 192 | |
'+' should be on a new line. | 193 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 210 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 63 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 31 | |
First sentence should end with a period. | 33 | |
First sentence should end with a period. | 35 | |
First sentence should end with a period. | 37 | |
Parameter aggregatorClass should be final. | 47 | |
Parameter persistent should be final. | 48 | |
'persistent' hides a field. | 48 | |
Expected @throws tag for 'IllegalAccessException'. | 48 | |
Expected @throws tag for 'InstantiationException'. | 49 | |
First sentence should end with a period. | 56 | |
Method 'getPreviousAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 61 | |
First sentence should end with a period. | 65 | |
Method 'setPreviousAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 70 | |
Parameter value should be final. | 70 | |
First sentence should end with a period. | 74 | |
Method 'isPersistent' is not designed for extension - needs to be abstract, final or empty. | 79 | |
First sentence should end with a period. | 83 | |
Method 'isChanged' is not designed for extension - needs to be abstract, final or empty. | 88 | |
First sentence should end with a period. | 92 | |
Method 'aggregateCurrent' is not designed for extension - needs to be abstract, final or empty. | 97 | |
Parameter value should be final. | 97 | |
First sentence should end with a period. | 102 | |
Method 'getCurrentAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 107 | |
First sentence should end with a period. | 111 | |
Method 'setCurrentAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 116 | |
Parameter value should be final. | 116 | |
First sentence should end with a period. | 121 | |
Method 'resetCurrentAggregator' is not designed for extension - needs to be abstract, final or empty. | 124 | |
First sentence should end with a period. | 129 | |
Method 'createInitialValue' is not designed for extension - needs to be abstract, final or empty. | 134 | |
First sentence should end with a period. | 138 | |
Method 'getAggregatorClass' is not designed for extension - needs to be abstract, final or empty. | 143 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 35 | |
Expected @throws tag for 'IOException'. | 49 | |
Expected @throws tag for 'IOException'. | 62 | |
Expected @throws tag for 'IOException'. | 72 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 66 | |
First sentence should end with a period. | 68 | |
First sentence should end with a period. | 70 | |
First sentence should end with a period. | 72 | |
First sentence should end with a period. | 74 | |
First sentence should end with a period. | 76 | |
First sentence should end with a period. | 78 | |
First sentence should end with a period. | 80 | |
First sentence should end with a period. | 83 | |
First sentence should end with a period. | 86 | |
First sentence should end with a period. | 92 | |
First sentence should end with a period. | 95 | |
First sentence should end with a period. | 97 | |
First sentence should end with a period. | 99 | |
First sentence should end with a period. | 101 | |
First sentence should end with a period. | 108 | |
First sentence should end with a period. | 111 | |
First sentence should end with a period. | 113 | |
First sentence should end with a period. | 116 | |
First sentence should end with a period. | 119 | |
First sentence should end with a period. | 121 | |
First sentence should end with a period. | 123 | |
First sentence should end with a period. | 126 | |
First sentence should end with a period. | 129 | |
First sentence should end with a period. | 132 | |
First sentence should end with a period. | 134 | |
First sentence should end with a period. | 136 | |
First sentence should end with a period. | 138 | |
First sentence should end with a period. | 140 | |
First sentence should end with a period. | 143 | |
First sentence should end with a period. | 145 | |
First sentence should end with a period. | 147 | |
First sentence should end with a period. | 150 | |
First sentence should end with a period. | 153 | |
First sentence should end with a period. | 155 | |
First sentence should end with a period. | 157 | |
First sentence should end with a period. | 160 | |
First sentence should end with a period. | 163 | |
First sentence should end with a period. | 165 | |
First sentence should end with a period. | 168 | |
First sentence should end with a period. | 170 | |
First sentence should end with a period. | 172 | |
First sentence should end with a period. | 174 | |
First sentence should end with a period. | 181 | |
First sentence should end with a period. | 191 | |
First sentence should end with a period. | 193 | |
Variable 'basePath' must be private and have accessor methods. | 194 | |
First sentence should end with a period. | 195 | |
Variable 'masterJobStatePath' must be private and have accessor methods. | 196 | |
First sentence should end with a period. | 197 | |
Variable 'inputSplitsPath' must be private and have accessor methods. | 198 | |
First sentence should end with a period. | 199 | |
Variable 'inputSplitsAllReadyPath' must be private and have accessor methods. | 200 | |
First sentence should end with a period. | 201 | |
Variable 'inputSplitsDonePath' must be private and have accessor methods. | 202 | |
First sentence should end with a period. | 203 | |
Variable 'inputSplitsAllDonePath' must be private and have accessor methods. | 204 | |
First sentence should end with a period. | 205 | |
Variable 'applicationAttemptsPath' must be private and have accessor methods. | 206 | |
First sentence should end with a period. | 207 | |
Variable 'cleanedUpPath' must be private and have accessor methods. | 208 | |
First sentence should end with a period. | 209 | |
Variable 'checkpointBasePath' must be private and have accessor methods. | 210 | |
First sentence should end with a period. | 211 | |
Variable 'masterElectionPath' must be private and have accessor methods. | 212 | |
First sentence should end with a period. | 213 | |
Variable 'currentMasterPath' must be private and have accessor methods. | 214 | |
First sentence should end with a period. | 215 | |
First sentence should end with a period. | 219 | |
First sentence should end with a period. | 221 | |
First sentence should end with a period. | 223 | |
First sentence should end with a period. | 225 | |
First sentence should end with a period. | 227 | |
First sentence should end with a period. | 231 | |
First sentence should end with a period. | 233 | |
First sentence should end with a period. | 235 | |
First sentence should end with a period. | 237 | |
First sentence should end with a period. | 239 | |
First sentence should end with a period. | 242 | |
First sentence should end with a period. | 244 | |
First sentence should end with a period. | 246 | |
First sentence should end with a period. | 248 | |
First sentence should end with a period. | 250 | |
First sentence should end with a period. | 252 | |
First sentence should end with a period. | 254 | |
First sentence should end with a period. | 256 | |
First sentence should end with a period. | 258 | |
First sentence should end with a period. | 260 | |
First sentence should end with a period. | 262 | |
First sentence should end with a period. | 264 | |
First sentence should end with a period. | 266 | |
First sentence should end with a period. | 268 | |
Parameter serverPortList should be final. | 280 | |
Parameter sessionMsecTimeout should be final. | 281 | |
Parameter context should be final. | 282 | |
'context' hides a field. | 282 | |
Parameter graphMapper should be final. | 283 | |
'graphMapper' hides a field. | 283 | |
'&&' should be on a new line. | 314 | |
'+' should be on a new line. | 317 | |
'+' should be on a new line. | 348 | |
First sentence should end with a period. | 361 | |
Parameter path should be final. | 367 | |
'+' should be on a new line. | 371 | |
'+' should be on a new line. | 375 | |
'+' should be on a new line. | 379 | |
'+' should be on a new line. | 383 | |
First sentence should end with a period. | 389 | |
Parameter path should be final. | 395 | |
'+' should be on a new line. | 399 | |
First sentence should end with a period. | 406 | |
Parameter attempt should be final. | 413 | |
First sentence should end with a period. | 417 | |
Parameter attempt should be final. | 425 | |
Parameter superstep should be final. | 426 | |
'+' should be on a new line. | 427 | |
First sentence should end with a period. | 431 | |
Parameter attempt should be final. | 439 | |
Parameter superstep should be final. | 440 | |
'+' should be on a new line. | 441 | |
First sentence should end with a period. | 445 | |
Parameter attempt should be final. | 453 | |
Parameter superstep should be final. | 454 | |
'+' should be on a new line. | 455 | |
First sentence should end with a period. | 459 | |
Parameter attempt should be final. | 467 | |
Parameter superstep should be final. | 467 | |
'+' should be on a new line. | 468 | |
First sentence should end with a period. | 472 | |
Parameter attempt should be final. | 479 | |
Parameter superstep should be final. | 480 | |
'+' should be on a new line. | 481 | |
First sentence should end with a period. | 485 | |
Parameter attempt should be final. | 492 | |
Parameter superstep should be final. | 493 | |
'+' should be on a new line. | 494 | |
Parameter attempt should be final. | 507 | |
Parameter superstep should be final. | 508 | |
Parameter workerInfo should be final. | 509 | |
'+' should be on a new line. | 510 | |
First sentence should end with a period. | 514 | |
Parameter attempt should be final. | 521 | |
Parameter superstep should be final. | 521 | |
'+' should be on a new line. | 522 | |
First sentence should end with a period. | 526 | |
Parameter attempt should be final. | 533 | |
Parameter superstep should be final. | 533 | |
'+' should be on a new line. | 534 | |
First sentence should end with a period. | 538 | |
Parameter superstep should be final. | 545 | |
First sentence should end with a period. | 549 | |
Parameter finalizedPath should be final. | 555 | |
'+' should be on a new line. | 558 | |
First sentence should end with a period. | 580 | |
Parameter superstep should be final. | 585 | |
Parameter superstep should be final. | 601 | |
First sentence should end with a period. | 615 | |
Missing a Javadoc comment. | 624 | |
Missing a Javadoc comment. | 628 | |
Missing a Javadoc comment. | 632 | |
Missing a Javadoc comment. | 636 | |
Missing a Javadoc comment. | 640 | |
Missing a Javadoc comment. | 644 | |
Missing a Javadoc comment. | 648 | |
Missing a Javadoc comment. | 652 | |
Missing a Javadoc comment. | 656 | |
Missing a Javadoc comment. | 660 | |
Missing a Javadoc comment. | 664 | |
Missing a Javadoc comment. | 668 | |
Missing a Javadoc comment. | 673 | |
Missing a Javadoc comment. | 677 | |
Missing a Javadoc comment. | 682 | |
Missing a Javadoc comment. | 686 | |
'+' should be on a new line. | 706 | |
'+' should be on a new line. | 709 | |
'+' should be on a new line. | 712 | |
'+' should be on a new line. | 727 | |
'+' should be on a new line. | 730 | |
'+' should be on a new line. | 733 | |
First sentence should end with a period. | 744 | |
'+' should be on a new line. | 769 | |
'+' should be on a new line. | 772 | |
'+' should be on a new line. | 775 | |
'+' should be on a new line. | 789 | |
'+' should be on a new line. | 792 | |
'+' should be on a new line. | 819 | |
'+' should be on a new line. | 857 | |
Parameter superstep should be final. | 869 | |
First sentence should end with a period. | 873 | |
Parameter applicationAttempt should be final. | 879 | |
'+' should be on a new line. | 890 | |
'+' should be on a new line. | 894 | |
'+' should be on a new line. | 898 | |
Method 'registerAggregator' is not designed for extension - needs to be abstract, final or empty. | 915 | |
Parameter name should be final. | 916 | |
Parameter aggregatorClass should be final. | 916 | |
Parameter persistent should be final. | 917 | |
Expected @throws tag for 'InstantiationException'. | 917 | |
Expected @throws tag for 'IllegalAccessException'. | 918 | |
Method 'getAggregator' is not designed for extension - needs to be abstract, final or empty. | 939 | |
Parameter name should be final. | 939 | |
Method 'getAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 950 | |
Parameter name should be final. | 950 | |
Method 'getAggregatorMap' is not designed for extension - needs to be abstract, final or empty. | 964 | |
Method 'registerBspEvent' is not designed for extension - needs to be abstract, final or empty. | 975 | |
Parameter event should be final. | 975 | |
First sentence should end with a period. | 979 | |
Method 'getGraphPartitionerFactory' is not designed for extension - needs to be abstract, final or empty. | 984 | |
Method 'processEvent' is not designed for extension - needs to be abstract, final or empty. | 995 | |
Parameter event should be final. | 995 | |
Parameter event should be final. | 1000 | |
'+' should be on a new line. | 1004 | |
'+' should be on a new line. | 1005 | |
'+' should be on a new line. | 1016 | |
'||' should be on a new line. | 1035 | |
'&&' should be on a new line. | 1036 | |
'+' should be on a new line. | 1039 | |
'&&' should be on a new line. | 1044 | |
'+' should be on a new line. | 1047 | |
'&&' should be on a new line. | 1052 | |
'+' should be on a new line. | 1055 | |
'&&' should be on a new line. | 1060 | |
'+' should be on a new line. | 1063 | |
'&&' should be on a new line. | 1068 | |
'+' should be on a new line. | 1071 | |
'&&' should be on a new line. | 1076 | |
'+' should be on a new line. | 1079 | |
'&&' should be on a new line. | 1084 | |
'+' should be on a new line. | 1087 | |
'&&' should be on a new line. | 1092 | |
'+' should be on a new line. | 1095 | |
'&&' should be on a new line. | 1100 | |
'&&' should be on a new line. | 1107 | |
'&&' should be on a new line. | 1114 | |
'&&' should be on a new line. | 1121 | |
'+' should be on a new line. | 1131 | |
'+' should be on a new line. | 1132 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 93 | |
First sentence should end with a period. | 95 | |
First sentence should end with a period. | 97 | |
First sentence should end with a period. | 99 | |
First sentence should end with a period. | 101 | |
First sentence should end with a period. | 103 | |
First sentence should end with a period. | 105 | |
First sentence should end with a period. | 107 | |
First sentence should end with a period. | 109 | |
First sentence should end with a period. | 111 | |
First sentence should end with a period. | 113 | |
First sentence should end with a period. | 117 | |
First sentence should end with a period. | 119 | |
First sentence should end with a period. | 121 | |
First sentence should end with a period. | 123 | |
First sentence should end with a period. | 125 | |
First sentence should end with a period. | 127 | |
First sentence should end with a period. | 129 | |
First sentence should end with a period. | 131 | |
First sentence should end with a period. | 133 | |
First sentence should end with a period. | 135 | |
First sentence should end with a period. | 137 | |
First sentence should end with a period. | 140 | |
First sentence should end with a period. | 142 | |
First sentence should end with a period. | 144 | |
First sentence should end with a period. | 146 | |
First sentence should end with a period. | 148 | |
Parameter serverPortList should be final. | 160 | |
Parameter sessionMsecTimeout should be final. | 161 | |
Parameter context should be final. | 162 | |
Parameter graphMapper should be final. | 163 | |
'100.0f' is a magic number. | 176 | |
Method 'setJobState' is not designed for extension - needs to be abstract, final or empty. | 190 | |
Parameter state should be final. | 191 | |
Parameter applicationAttempt should be final. | 192 | |
Parameter desiredSuperstep should be final. | 193 | |
'+' should be on a new line. | 200 | |
'+' should be on a new line. | 204 | |
'+' should be on a new line. | 215 | |
'+' should be on a new line. | 219 | |
'+' should be on a new line. | 223 | |
Parameter numWorkers should be final. | 243 | |
'100f' is a magic number. | 254 | |
'+' should be on a new line. | 256 | |
'+' should be on a new line. | 257 | |
'+' should be on a new line. | 262 | |
Parameter workerInfosPath should be final. | 307 | |
Parameter watch should be final. | 308 | |
First sentence should end with a period. | 330 | |
Parameter superstep should be final. | 339 | |
Parameter healthyWorkerInfoList should be final. | 340 | |
Parameter unhealthyWorkerInfoList should be final. | 341 | |
':' should be on a new line. | 378 | |
':' should be on a new line. | 386 | |
'+' should be on a new line. | 409 | |
'100.0f' is a magic number. | 411 | |
'>=' should be on a new line. | 411 | |
'+' should be on a new line. | 416 | |
'+' should be on a new line. | 417 | |
'+' should be on a new line. | 418 | |
'+' should be on a new line. | 419 | |
'+' should be on a new line. | 420 | |
'+' should be on a new line. | 425 | |
'+' should be on a new line. | 432 | |
'+' should be on a new line. | 433 | |
'+' should be on a new line. | 434 | |
'+' should be on a new line. | 435 | |
'+' should be on a new line. | 436 | |
'<=' should be on a new line. | 439 | |
'+' should be on a new line. | 454 | |
'+' should be on a new line. | 463 | |
'+' should be on a new line. | 464 | |
'+' should be on a new line. | 465 | |
'+' should be on a new line. | 466 | |
'+' should be on a new line. | 467 | |
'+' should be on a new line. | 473 | |
'+' should be on a new line. | 478 | |
'+' should be on a new line. | 479 | |
'+' should be on a new line. | 480 | |
Method 'createInputSplits' is not designed for extension - needs to be abstract, final or empty. | 486 | |
'+' should be on a new line. | 494 | |
'+' should be on a new line. | 502 | |
'+' should be on a new line. | 526 | |
'+' should be on a new line. | 527 | |
'+' should be on a new line. | 528 | |
'+' should be on a new line. | 529 | |
Avoid inline conditionals. | 551 | |
Avoid inline conditionals. | 555 | |
'+' should be on a new line. | 566 | |
'+' should be on a new line. | 567 | |
'+' should be on a new line. | 572 | |
'+' should be on a new line. | 594 | |
Parameter superstep should be final. | 622 | |
Parameter partitionOwners should be final. | 623 | |
Expected @throws tag for 'IOException'. | 624 | |
Expected @throws tag for 'KeeperException'. | 624 | |
Expected @throws tag for 'InterruptedException'. | 624 | |
'+' should be on a new line. | 649 | |
'+' should be on a new line. | 650 | |
'+' should be on a new line. | 656 | |
'+' should be on a new line. | 657 | |
'+' should be on a new line. | 658 | |
'+' should be on a new line. | 659 | |
'+' should be on a new line. | 681 | |
'-' should be on a new line. | 692 | |
'+' should be on a new line. | 701 | |
'+' should be on a new line. | 702 | |
'+' should be on a new line. | 703 | |
Method 'setup' is not designed for extension - needs to be abstract, final or empty. | 712 | |
Method 'becomeMaster' is not designed for extension - needs to be abstract, final or empty. | 739 | |
'+' should be on a new line. | 746 | |
'&&' should be on a new line. | 762 | |
'==' should be on a new line. | 764 | |
'+' should be on a new line. | 766 | |
'+' should be on a new line. | 780 | |
'+' should be on a new line. | 781 | |
'-' should be on a new line. | 786 | |
'+' should be on a new line. | 796 | |
Parameter superstep should be final. | 836 | |
'+' should be on a new line. | 892 | |
Parameter superstep should be final. | 903 | |
'+' should be on a new line. | 942 | |
'+' should be on a new line. | 960 | |
'+' should be on a new line. | 961 | |
'+' should be on a new line. | 970 | |
':' should be on a new line. | 980 | |
Parameter superstep should be final. | 996 | |
':' should be on a new line. | 1021 | |
'+' should be on a new line. | 1028 | |
'+' should be on a new line. | 1040 | |
'+' should be on a new line. | 1051 | |
Parameter superstep should be final. | 1068 | |
Parameter chosenWorkerInfoList should be final. | 1069 | |
Expected @throws tag for 'IOException'. | 1070 | |
Expected @throws tag for 'KeeperException'. | 1070 | |
Expected @throws tag for 'InterruptedException'. | 1070 | |
'+' should be on a new line. | 1072 | |
'+' should be on a new line. | 1077 | |
'+' should be on a new line. | 1098 | |
'-' should be on a new line. | 1115 | |
Parameter allPartitionStatsList should be final. | 1130 | |
'allPartitionStatsList' hides a field. | 1130 | |
Parameter chosenWorkerInfoList should be final. | 1131 | |
Parameter masterGraphPartitioner should be final. | 1132 | |
'masterGraphPartitioner' hides a field. | 1132 | |
'||' should be on a new line. | 1134 | |
'+' should be on a new line. | 1185 | |
'+' should be on a new line. | 1189 | |
First sentence should end with a period. | 1205 | |
Parameter chosenWorkerInfoHealthPath should be final. | 1215 | |
Parameter chosenWorkerInfoList should be final. | 1216 | |
Expected @throws tag for 'KeeperException'. | 1217 | |
Expected @throws tag for 'InterruptedException'. | 1217 | |
'+' should be on a new line. | 1226 | |
'+' should be on a new line. | 1227 | |
Method 'restartFromCheckpoint' is not designed for extension - needs to be abstract, final or empty. | 1234 | |
Parameter checkpoint should be final. | 1235 | |
First sentence should end with a period. | 1257 | |
Method 'accept' is not designed for extension - needs to be abstract, final or empty. | 1261 | |
Parameter path should be final. | 1262 | |
Method 'getLastGoodCheckpoint' is not designed for extension - needs to be abstract, final or empty. | 1267 | |
'+' should be on a new line. | 1282 | |
'+' should be on a new line. | 1283 | |
Parameter finishedWorkerPath should be final. | 1302 | |
Parameter workerInfoList should be final. | 1303 | |
Parameter event should be final. | 1304 | |
'+' should be on a new line. | 1313 | |
'+' should be on a new line. | 1317 | |
'+' should be on a new line. | 1338 | |
'+' should be on a new line. | 1342 | |
'+' should be on a new line. | 1346 | |
'+' should be on a new line. | 1347 | |
'+' should be on a new line. | 1348 | |
'+' should be on a new line. | 1349 | |
'+' should be on a new line. | 1350 | |
'+' should be on a new line. | 1351 | |
'&&' should be on a new line. | 1355 | |
'30000' is a magic number. | 1357 | |
'+' should be on a new line. | 1358 | |
'+' should be on a new line. | 1359 | |
'+' should be on a new line. | 1360 | |
'+' should be on a new line. | 1361 | |
'<' should be on a new line. | 1363 | |
'+' should be on a new line. | 1370 | |
'+' should be on a new line. | 1371 | |
'+' should be on a new line. | 1372 | |
'+' should be on a new line. | 1373 | |
'30' is a magic number. | 1381 | |
'1000' is a magic number. | 1381 | |
'&&' should be on a new line. | 1387 | |
'+' should be on a new line. | 1395 | |
'+' should be on a new line. | 1399 | |
First sentence should end with a period. | 1407 | |
Parameter removeableSuperstep should be final. | 1413 | |
Expected @throws tag for 'InterruptedException'. | 1414 | |
'&&' should be on a new line. | 1417 | |
'+' should be on a new line. | 1420 | |
'+' should be on a new line. | 1424 | |
'+' should be on a new line. | 1431 | |
'+' should be on a new line. | 1435 | |
Method 'coordinateSuperstep' is not designed for extension - needs to be abstract, final or empty. | 1441 | |
Method length is 157 lines (max allowed is 150). | 1441 | |
'+' should be on a new line. | 1454 | |
'+' should be on a new line. | 1461 | |
'+' should be on a new line. | 1464 | |
'+' should be on a new line. | 1465 | |
'-' should be on a new line. | 1477 | |
'+' should be on a new line. | 1511 | |
'+' should be on a new line. | 1520 | |
'||' should be on a new line. | 1548 | |
'==' should be on a new line. | 1549 | |
'&&' should be on a new line. | 1550 | |
Parameter entry should be final. | 1583 | |
'?' should be on a new line. | 1590 | |
Avoid inline conditionals. | 1590 | |
'+' should be on a new line. | 1594 | |
First sentence should end with a period. | 1601 | |
Parameter superstep should be final. | 1606 | |
'+' should be on a new line. | 1645 | |
'||' should be on a new line. | 1657 | |
'==' should be on a new line. | 1658 | |
'+' should be on a new line. | 1669 | |
'+' should be on a new line. | 1670 | |
'+' should be on a new line. | 1671 | |
'+' should be on a new line. | 1678 | |
'+' should be on a new line. | 1679 | |
'+' should be on a new line. | 1680 | |
'+' should be on a new line. | 1685 | |
'+' should be on a new line. | 1690 | |
'+' should be on a new line. | 1706 | |
'+' should be on a new line. | 1712 | |
'+' should be on a new line. | 1715 | |
Method 'cleanup' is not designed for extension - needs to be abstract, final or empty. | 1720 | |
'+' should be on a new line. | 1726 | |
'+' should be on a new line. | 1736 | |
'+' should be on a new line. | 1741 | |
'+' should be on a new line. | 1759 | |
'+' should be on a new line. | 1760 | |
First sentence should end with a period. | 1782 | |
First sentence should end with a period. | 1791 | |
Parameter failedWorkerPath should be final. | 1806 | |
'||' should be on a new line. | 1819 | |
'&&' should be on a new line. | 1820 | |
'+' should be on a new line. | 1822 | |
'+' should be on a new line. | 1823 | |
'+' should be on a new line. | 1824 | |
'+' should be on a new line. | 1825 | |
'+' should be on a new line. | 1826 | |
Method 'processEvent' is not designed for extension - needs to be abstract, final or empty. | 1833 | |
Parameter event should be final. | 1834 | |
'&&' should be on a new line. | 1836 | |
'+' should be on a new line. | 1839 | |
'&&' should be on a new line. | 1845 | |
'+' should be on a new line. | 1848 | |
'&&' should be on a new line. | 1853 | |
'+' should be on a new line. | 1856 | |
Method 'registerAggregator' is not designed for extension - needs to be abstract, final or empty. | 1866 | |
Parameter name should be final. | 1867 | |
Parameter aggregatorClass should be final. | 1868 | |
Method 'registerPersistentAggregator' is not designed for extension - needs to be abstract, final or empty. | 1873 | |
Parameter name should be final. | 1874 | |
Parameter aggregatorClass should be final. | 1875 | |
Method 'setAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 1880 | |
Parameter name should be final. | 1881 | |
Parameter value should be final. | 1881 | |
'+' should be on a new line. | 1885 | |
First sentence should end with a period. | 1891 | |
Parameter globalStats should be final. | 1896 | |
'-' should be on a new line. | 1898 | |
'-' should be on a new line. | 1901 | |
'-' should be on a new line. | 1904 | |
'-' should be on a new line. | 1907 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 89 | |
First sentence should end with a period. | 91 | |
First sentence should end with a period. | 93 | |
First sentence should end with a period. | 95 | |
First sentence should end with a period. | 97 | |
First sentence should end with a period. | 99 | |
First sentence should end with a period. | 102 | |
First sentence should end with a period. | 104 | |
First sentence should end with a period. | 113 | |
First sentence should end with a period. | 115 | |
First sentence should end with a period. | 117 | |
First sentence should end with a period. | 119 | |
Parameter serverPortList should be final. | 138 | |
Parameter sessionMsecTimeout should be final. | 139 | |
Parameter context should be final. | 140 | |
Parameter graphMapper should be final. | 141 | |
Parameter graphState should be final. | 142 | |
Expected @throws tag for 'IOException'. | 143 | |
Expected @throws tag for 'InterruptedException'. | 143 | |
'+' should be on a new line. | 164 | |
'+' should be on a new line. | 166 | |
'+' should be on a new line. | 167 | |
Method 'getWorkerContext' is not designed for extension - needs to be abstract, final or empty. | 187 | |
Missing a Javadoc comment. | 187 | |
Comment matches to-do format 'TODO:'. | 194 | |
Method 'isHealthy' is not designed for extension - needs to be abstract, final or empty. | 199 | |
Expected @throws tag for 'KeeperException'. | 221 | |
Expected @throws tag for 'InterruptedException'. | 221 | |
'100.0f' is a magic number. | 252 | |
'/' should be on a new line. | 252 | |
'+' should be on a new line. | 254 | |
'+' should be on a new line. | 255 | |
'+' should be on a new line. | 256 | |
'+' is followed by whitespace. | 257 | |
'+' should be on a new line. | 257 | |
'+' should be on a new line. | 262 | |
'+' should be on a new line. | 263 | |
'+' should be on a new line. | 270 | |
'+' should be on a new line. | 278 | |
'+' should be on a new line. | 279 | |
'+' should be on a new line. | 280 | |
'60' is a magic number. | 290 | |
'1000' is a magic number. | 290 | |
Expected @throws tag for 'IOException'. | 312 | |
Expected @throws tag for 'ClassNotFoundException'. | 313 | |
Expected @throws tag for 'InterruptedException'. | 313 | |
Expected @throws tag for 'InstantiationException'. | 313 | |
Expected @throws tag for 'IllegalAccessException'. | 314 | |
Expected @throws tag for 'KeeperException'. | 314 | |
':' should be on a new line. | 323 | |
Parameter inputSplitPath should be final. | 344 | |
'+' should be on a new line. | 354 | |
'+' should be on a new line. | 358 | |
'+' should be on a new line. | 362 | |
Parameter inputSplitPath should be final. | 381 | |
Expected @throws tag for 'IOException'. | 382 | |
Expected @throws tag for 'ClassNotFoundException'. | 382 | |
Expected @throws tag for 'InterruptedException'. | 382 | |
Expected @throws tag for 'InstantiationException'. | 383 | |
Expected @throws tag for 'IllegalAccessException'. | 383 | |
'+' should be on a new line. | 388 | |
Parameter inputSplitPath should be final. | 404 | |
Expected @throws tag for 'IOException'. | 405 | |
Expected @throws tag for 'ClassNotFoundException'. | 405 | |
'+' should be on a new line. | 429 | |
'+' should be on a new line. | 430 | |
Parameter inputSplit should be final. | 446 | |
Expected @throws tag for 'IOException'. | 446 | |
Expected @throws tag for 'InterruptedException'. | 446 | |
'+' should be on a new line. | 458 | |
'+' should be on a new line. | 479 | |
'250000' is a magic number. | 493 | |
'+' should be on a new line. | 494 | |
'+' should be on a new line. | 495 | |
'+' should be on a new line. | 496 | |
'+' should be on a new line. | 497 | |
'+' should be on a new line. | 498 | |
'+' should be on a new line. | 499 | |
'&&' should be on a new line. | 509 | |
'>=' should be on a new line. | 510 | |
'+' should be on a new line. | 513 | |
'+' should be on a new line. | 514 | |
Method 'assignMessagesToVertex' is not designed for extension - needs to be abstract, final or empty. | 526 | |
Parameter vertex should be final. | 527 | |
Parameter messages should be final. | 528 | |
Method 'getMasterInfo' is not designed for extension - needs to be abstract, final or empty. | 532 | |
Method 'setup' is not designed for extension - needs to be abstract, final or empty. | 537 | |
Method length is 158 lines (max allowed is 150). | 537 | |
'==' should be on a new line. | 553 | |
'&&' should be on a new line. | 554 | |
'==' should be on a new line. | 555 | |
'+' should be on a new line. | 558 | |
'+' should be on a new line. | 559 | |
'+' should be on a new line. | 560 | |
'+' should be on a new line. | 568 | |
'+' should be on a new line. | 606 | |
'+' should be on a new line. | 610 | |
'+' should be on a new line. | 613 | |
'+' should be on a new line. | 616 | |
'+' should be on a new line. | 619 | |
'+' should be on a new line. | 622 | |
'+' should be on a new line. | 625 | |
'+' should be on a new line. | 656 | |
'&&' should be on a new line. | 668 | |
':' should be on a new line. | 682 | |
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 697 | |
Parameter name should be final. | 698 | |
Parameter value should be final. | 698 | |
'+' should be on a new line. | 703 | |
Parameter superstep should be final. | 715 | |
':' should be on a new line. | 722 | |
'+' should be on a new line. | 729 | |
Parameter superstep should be final. | 747 | |
':' should be on a new line. | 749 | |
'+' should be on a new line. | 761 | |
'+' should be on a new line. | 765 | |
'+' should be on a new line. | 768 | |
'+' should be on a new line. | 779 | |
'+' should be on a new line. | 797 | |
'+' should be on a new line. | 798 | |
'+' should be on a new line. | 801 | |
'+' should be on a new line. | 802 | |
'+' should be on a new line. | 805 | |
'+' should be on a new line. | 806 | |
'+' should be on a new line. | 820 | |
First sentence should end with a period. | 825 | |
Parameter superstep should be final. | 830 | |
'+' should be on a new line. | 853 | |
'+' should be on a new line. | 854 | |
'+' should be on a new line. | 855 | |
'+' should be on a new line. | 859 | |
'+' should be on a new line. | 863 | |
'+' should be on a new line. | 866 | |
'+' should be on a new line. | 869 | |
'+' should be on a new line. | 873 | |
'+' should be on a new line. | 874 | |
'+' should be on a new line. | 875 | |
'+' should be on a new line. | 884 | |
'+' should be on a new line. | 890 | |
'+' should be on a new line. | 894 | |
Method 'failureCleanup' is not designed for extension - needs to be abstract, final or empty. | 899 | |
Method 'startSuperstep' is not designed for extension - needs to be abstract, final or empty. | 904 | |
'==' should be on a new line. | 923 | |
'+' should be on a new line. | 959 | |
'+' should be on a new line. | 960 | |
'+' should be on a new line. | 961 | |
'+' should be on a new line. | 968 | |
'+' should be on a new line. | 969 | |
'+' should be on a new line. | 970 | |
Method 'finishSuperstep' is not designed for extension - needs to be abstract, final or empty. | 975 | |
Parameter partitionStatsList should be final. | 976 | |
'+' should be on a new line. | 990 | |
'+' should be on a new line. | 991 | |
'+' should be on a new line. | 992 | |
'+' should be on a new line. | 1010 | |
'+' should be on a new line. | 1011 | |
'+' should be on a new line. | 1036 | |
'+' should be on a new line. | 1045 | |
'+' should be on a new line. | 1048 | |
'+' should be on a new line. | 1051 | |
'+' should be on a new line. | 1054 | |
'+' should be on a new line. | 1055 | |
'+' should be on a new line. | 1056 | |
'+' should be on a new line. | 1057 | |
'+' should be on a new line. | 1069 | |
'+' should be on a new line. | 1073 | |
'+' should be on a new line. | 1080 | |
'+' should be on a new line. | 1084 | |
'+' should be on a new line. | 1085 | |
'+' should be on a new line. | 1086 | |
Expected @throws tag for 'IOException'. | 1099 | |
Expected @throws tag for 'InterruptedException'. | 1099 | |
'==' should be on a new line. | 1100 | |
'+' should be on a new line. | 1102 | |
':' should be on a new line. | 1112 | |
Method 'cleanup' is not designed for extension - needs to be abstract, final or empty. | 1121 | |
'+' should be on a new line. | 1130 | |
'+' should be on a new line. | 1140 | |
'+' should be on a new line. | 1145 | |
'+' should be on a new line. | 1150 | |
'+' should be on a new line. | 1154 | |
Method 'storeCheckpoint' is not designed for extension - needs to be abstract, final or empty. | 1170 | |
'+' should be on a new line. | 1172 | |
'+' should be on a new line. | 1173 | |
'+' should be on a new line. | 1174 | |
'+' should be on a new line. | 1180 | |
'+' should be on a new line. | 1181 | |
'+' should be on a new line. | 1184 | |
'+' should be on a new line. | 1185 | |
'+' should be on a new line. | 1188 | |
'+' should be on a new line. | 1189 | |
'+' should be on a new line. | 1195 | |
'+' should be on a new line. | 1199 | |
':' should be on a new line. | 1212 | |
'+' should be on a new line. | 1230 | |
'+' should be on a new line. | 1231 | |
'+' should be on a new line. | 1232 | |
'+' should be on a new line. | 1245 | |
'+' should be on a new line. | 1262 | |
'+' should be on a new line. | 1265 | |
'+' should be on a new line. | 1268 | |
Method 'loadCheckpoint' is not designed for extension - needs to be abstract, final or empty. | 1273 | |
Parameter superstep should be final. | 1274 | |
':' should be on a new line. | 1293 | |
'+' should be on a new line. | 1297 | |
'+' should be on a new line. | 1300 | |
'+' should be on a new line. | 1316 | |
'+' should be on a new line. | 1328 | |
'+' should be on a new line. | 1338 | |
'+' should be on a new line. | 1343 | |
'+' should be on a new line. | 1350 | |
'+' should be on a new line. | 1356 | |
'+' should be on a new line. | 1357 | |
'+' should be on a new line. | 1358 | |
First sentence should end with a period. | 1383 | |
Parameter workerPartitionMap should be final. | 1390 | |
':' should be on a new line. | 1395 | |
'+' should be on a new line. | 1402 | |
'+' should be on a new line. | 1403 | |
'+' should be on a new line. | 1407 | |
'+' should be on a new line. | 1408 | |
'+' should be on a new line. | 1433 | |
'+' should be on a new line. | 1437 | |
Parameter masterSetPartitionOwners should be final. | 1447 | |
'+' should be on a new line. | 1476 | |
'+' should be on a new line. | 1494 | |
Method 'processEvent' is not designed for extension - needs to be abstract, final or empty. | 1520 | |
Parameter event should be final. | 1521 | |
'&&' should be on a new line. | 1523 | |
'+' should be on a new line. | 1526 | |
'==' should be on a new line. | 1531 | |
'&&' should be on a new line. | 1532 | |
'!=' should be on a new line. | 1533 | |
'+' should be on a new line. | 1535 | |
'+' should be on a new line. | 1541 | |
'&&' should be on a new line. | 1545 | |
'+' should be on a new line. | 1548 | |
Method 'getWorkerInfo' is not designed for extension - needs to be abstract, final or empty. | 1558 | |
Method 'getPartitionStore' is not designed for extension - needs to be abstract, final or empty. | 1563 | |
Method 'getPartitionOwners' is not designed for extension - needs to be abstract, final or empty. | 1572 | |
Method 'getVertexPartitionOwner' is not designed for extension - needs to be abstract, final or empty. | 1577 | |
Parameter vertexId should be final. | 1578 | |
Method 'getPartition' is not designed for extension - needs to be abstract, final or empty. | 1582 | |
Parameter vertexId should be final. | 1583 | |
Method 'getPartitionId' is not designed for extension - needs to be abstract, final or empty. | 1587 | |
Parameter vertexId should be final. | 1588 | |
Method 'hasPartition' is not designed for extension - needs to be abstract, final or empty. | 1593 | |
Parameter partitionId should be final. | 1594 | |
Method 'getVertex' is not designed for extension - needs to be abstract, final or empty. | 1598 | |
Parameter vertexId should be final. | 1599 | |
Method 'getServerData' is not designed for extension - needs to be abstract, final or empty. | 1609 |
Violation | Message | Line |
---|---|---|
Class BspUtils should be declared as final. | 34 | |
Parameter conf should be final. | 54 | |
First sentence should end with a period. | 61 | |
Parameter conf should be final. | 75 | |
First sentence should end with a period. | 81 | |
Parameter conf should be final. | 94 | |
Parameter conf should be final. | 117 | |
First sentence should end with a period. | 124 | |
Parameter conf should be final. | 139 | |
Parameter conf should be final. | 161 | |
First sentence should end with a period. | 168 | |
Parameter conf should be final. | 180 | |
Parameter conf should be final. | 193 | |
First sentence should end with a period. | 199 | |
Parameter conf should be final. | 205 | |
Parameter conf should be final. | 222 | |
First sentence should end with a period. | 229 | |
Parameter conf should be final. | 239 | |
Parameter conf should be final. | 260 | |
First sentence should end with a period. | 267 | |
Parameter conf should be final. | 281 | |
Parameter graphState should be final. | 282 | |
Parameter conf should be final. | 298 | |
First sentence should end with a period. | 305 | |
Parameter conf should be final. | 319 | |
Parameter graphState should be final. | 320 | |
First sentence should end with a period. | 329 | |
Parameter conf should be final. | 336 | |
First sentence should end with a period. | 343 | |
Parameter conf should be final. | 350 | |
First sentence should end with a period. | 358 | |
Parameter conf should be final. | 371 | |
First sentence should end with a period. | 378 | |
Parameter conf should be final. | 391 | |
Parameter conf should be final. | 407 | |
First sentence should end with a period. | 412 | |
Parameter conf should be final. | 421 | |
Parameter conf should be final. | 443 | |
First sentence should end with a period. | 448 | |
Parameter conf should be final. | 457 | |
Parameter conf should be final. | 483 | |
First sentence should end with a period. | 488 | |
Parameter conf should be final. | 497 | |
Parameter conf should be final. | 523 | |
First sentence should end with a period. | 528 | |
Parameter conf should be final. | 537 |
Violation | Message | Line |
---|---|---|
Parameter in should be final. | 34 | |
Parameter out should be final. | 38 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 34 | |
First sentence should end with a period. | 36 | |
First sentence should end with a period. | 39 | |
First sentence should end with a period. | 44 | |
Parameter targetVertexId should be final. | 50 | |
'targetVertexId' hides a field. | 50 | |
Parameter value should be final. | 50 | |
'value' hides a field. | 50 | |
First sentence should end with a period. | 55 | |
Method 'getTargetVertexId' is not designed for extension - needs to be abstract, final or empty. | 60 | |
First sentence should end with a period. | 64 | |
Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 69 | |
Method 'setTargetVertexId' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Parameter targetVertexId should be final. | 78 | |
'targetVertexId' hides a field. | 78 | |
Method 'setValue' is not designed for extension - needs to be abstract, final or empty. | 87 | |
Parameter value should be final. | 87 | |
'value' hides a field. | 87 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 91 | |
'+' should be on a new line. | 93 | |
Method 'compareTo' is not designed for extension - needs to be abstract, final or empty. | 97 | |
Parameter edge should be final. | 99 | |
Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 103 | |
Parameter o should be final. | 104 | |
Avoid inline conditionals. | 114 | |
':' should be on a new line. | 114 | |
Avoid inline conditionals. | 118 | |
Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 125 | |
Avoid inline conditionals. | 127 | |
'31' is a magic number. | 128 | |
Avoid inline conditionals. | 128 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 49 | |
First sentence should end with a period. | 51 | |
First sentence should end with a period. | 53 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 56 | |
Parameter id should be final. | 57 | |
Parameter value should be final. | 57 | |
Parameter edges should be final. | 57 | |
Parameter messages should be final. | 57 | |
Method 'getEdges' is not designed for extension - needs to be abstract, final or empty. | 69 | |
Parameter targetVertexId should be final. | 75 | |
Parameter value should be final. | 75 | |
'+' should be on a new line. | 78 | |
'+' should be on a new line. | 79 | |
Method 'getNumEdges' is not designed for extension - needs to be abstract, final or empty. | 88 | |
Method 'removeEdge' is not designed for extension - needs to be abstract, final or empty. | 93 | |
Parameter targetVertexId should be final. | 94 | |
Method 'putMessages' is not designed for extension - needs to be abstract, final or empty. | 106 | |
Parameter messages should be final. | 107 | |
Method 'getMessages' is not designed for extension - needs to be abstract, final or empty. | 112 | |
Method 'getNumMessages' is not designed for extension - needs to be abstract, final or empty. | 117 | |
Parameter in should be final. | 123 | |
Parameter out should be final. | 157 | |
Method 'releaseResources' is not designed for extension - needs to be abstract, final or empty. | 175 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 41 | |
First sentence should end with a period. | 43 | |
First sentence should end with a period. | 47 | |
First sentence should end with a period. | 50 | |
First sentence should end with a period. | 53 | |
First sentence should end with a period. | 56 | |
First sentence should end with a period. | 59 | |
First sentence should end with a period. | 63 | |
First sentence should end with a period. | 65 | |
First sentence should end with a period. | 67 | |
First sentence should end with a period. | 69 | |
First sentence should end with a period. | 71 | |
First sentence should end with a period. | 74 | |
First sentence should end with a period. | 78 | |
First sentence should end with a period. | 99 | |
First sentence should end with a period. | 103 | |
First sentence should end with a period. | 112 | |
First sentence should end with a period. | 115 | |
First sentence should end with a period. | 117 | |
First sentence should end with a period. | 120 | |
First sentence should end with a period. | 126 | |
First sentence should end with a period. | 129 | |
First sentence should end with a period. | 132 | |
First sentence should end with a period. | 135 | |
First sentence should end with a period. | 139 | |
First sentence should end with a period. | 142 | |
First sentence should end with a period. | 145 | |
First sentence should end with a period. | 148 | |
First sentence should end with a period. | 151 | |
First sentence should end with a period. | 154 | |
First sentence should end with a period. | 157 | |
First sentence should end with a period. | 159 | |
First sentence should end with a period. | 162 | |
First sentence should end with a period. | 164 | |
First sentence should end with a period. | 170 | |
First sentence should end with a period. | 173 | |
First sentence should end with a period. | 177 | |
First sentence should end with a period. | 180 | |
First sentence should end with a period. | 184 | |
First sentence should end with a period. | 187 | |
First sentence should end with a period. | 190 | |
First sentence should end with a period. | 193 | |
First sentence should end with a period. | 196 | |
First sentence should end with a period. | 199 | |
First sentence should end with a period. | 202 | |
First sentence should end with a period. | 205 | |
First sentence should end with a period. | 208 | |
First sentence should end with a period. | 211 | |
First sentence should end with a period. | 214 | |
First sentence should end with a period. | 216 | |
First sentence should end with a period. | 219 | |
First sentence should end with a period. | 222 | |
First sentence should end with a period. | 224 | |
First sentence should end with a period. | 230 | |
First sentence should end with a period. | 233 | |
First sentence should end with a period. | 235 | |
First sentence should end with a period. | 238 | |
First sentence should end with a period. | 241 | |
First sentence should end with a period. | 244 | |
First sentence should end with a period. | 247 | |
First sentence should end with a period. | 250 | |
First sentence should end with a period. | 253 | |
First sentence should end with a period. | 256 | |
First sentence should end with a period. | 259 | |
First sentence should end with a period. | 262 | |
First sentence should end with a period. | 264 | |
First sentence should end with a period. | 267 | |
First sentence should end with a period. | 270 | |
First sentence should end with a period. | 273 | |
First sentence should end with a period. | 276 | |
First sentence should end with a period. | 279 | |
First sentence should end with a period. | 282 | |
First sentence should end with a period. | 285 | |
First sentence should end with a period. | 289 | |
First sentence should end with a period. | 291 | |
First sentence should end with a period. | 294 | |
First sentence should end with a period. | 297 | |
First sentence should end with a period. | 300 | |
First sentence should end with a period. | 303 | |
First sentence should end with a period. | 309 | |
First sentence should end with a period. | 321 | |
First sentence should end with a period. | 332 | |
First sentence should end with a period. | 342 | |
First sentence should end with a period. | 348 | |
First sentence should end with a period. | 351 | |
'+' should be on a new line. | 353 | |
First sentence should end with a period. | 371 | |
First sentence should end with a period. | 388 | |
First sentence should end with a period. | 420 | |
First sentence should end with a period. | 423 | |
First sentence should end with a period. | 478 | |
First sentence should end with a period. | 481 | |
First sentence should end with a period. | 483 | |
First sentence should end with a period. | 487 | |
Parameter jobName should be final. | 493 | |
Expected @throws tag for 'IOException'. | 493 | |
Parameter conf should be final. | 504 | |
'conf' hides a field. | 504 | |
Parameter jobName should be final. | 504 | |
Expected @throws tag for 'IOException'. | 504 | |
Method 'getConfiguration' is not designed for extension - needs to be abstract, final or empty. | 514 | |
Method 'getInternalJob' is not designed for extension - needs to be abstract, final or empty. | 525 | |
'||' should be on a new line. | 537 | |
'100.0f' is a magic number. | 539 | |
'+' should be on a new line. | 541 | |
'+' should be on a new line. | 543 | |
'+' should be on a new line. | 559 | |
'+' should be on a new line. | 560 | |
First sentence should end with a period. | 566 | |
Parameter vertexClass should be final. | 571 | |
First sentence should end with a period. | 575 | |
Parameter vertexInputFormatClass should be final. | 581 | |
First sentence should end with a period. | 587 | |
Parameter masterComputeClass should be final. | 592 | |
First sentence should end with a period. | 597 | |
Parameter vertexOutputFormatClass should be final. | 603 | |
First sentence should end with a period. | 609 | |
Parameter vertexCombinerClass should be final. | 614 | |
First sentence should end with a period. | 620 | |
Parameter graphPartitionerFactoryClass should be final. | 626 | |
First sentence should end with a period. | 632 | |
Parameter vertexResolverClass should be final. | 637 | |
First sentence should end with a period. | 643 | |
Parameter workerContextClass should be final. | 649 | |
First sentence should end with a period. | 655 | |
Parameter aggregatorWriterClass should be final. | 662 | |
Parameter minWorkers should be final. | 678 | |
Parameter maxWorkers should be final. | 679 | |
Parameter minPercentResponded should be final. | 680 | |
Parameter serverList should be final. | 693 | |
Parameter conf should be final. | 704 | |
'+' should be on a new line. | 714 | |
'+' should be on a new line. | 715 | |
'+' should be on a new line. | 721 | |
'+' should be on a new line. | 722 | |
Parameter param should be final. | 733 | |
Parameter defaultValue should be final. | 733 | |
Parameter verbose should be final. | 748 | |
Expected @throws tag for 'IOException'. | 749 | |
Expected @throws tag for 'InterruptedException'. | 749 | |
Expected @throws tag for 'ClassNotFoundException'. | 749 | |
'512' is a magic number. | 754 | |
'1024' is a magic number. | 758 | |
'1024' is a magic number. | 759 | |
'60000' is a magic number. | 766 | |
'5' is a magic number. | 766 |
Violation | Message | Line |
---|---|---|
Comment matches to-do format 'TODO:'. | 35 | |
First sentence should end with a period. | 58 | |
First sentence should end with a period. | 66 | |
First sentence should end with a period. | 74 | |
First sentence should end with a period. | 77 | |
First sentence should end with a period. | 80 | |
First sentence should end with a period. | 83 | |
First sentence should end with a period. | 86 | |
Parameter conf should be final. | 89 | |
Method 'transferThisPartition' is not designed for extension - needs to be abstract, final or empty. | 107 | |
Parameter owner should be final. | 107 | |
'>=' should be on a new line. | 109 | |
'||' should be on a new line. | 110 | |
'>=' should be on a new line. | 111 | |
'?' should be on a new line. | 126 | |
Avoid inline conditionals. | 126 | |
'?' should be on a new line. | 136 | |
Avoid inline conditionals. | 136 | |
First sentence should end with a period. | 140 | |
Method 'clearCounters' is not designed for extension - needs to be abstract, final or empty. | 141 | |
Method 'incrementCounters' is not designed for extension - needs to be abstract, final or empty. | 160 | |
Parameter partitionOwner should be final. | 162 | |
Parameter vertex should be final. | 163 | |
'+' should be on a new line. | 171 | |
First sentence should end with a period. | 175 | |
Method 'getMaxEdgesPerTransfer' is not designed for extension - needs to be abstract, final or empty. | 177 | |
First sentence should end with a period. | 181 | |
Method 'getMaxVerticesPerTransfer' is not designed for extension - needs to be abstract, final or empty. | 183 | |
First sentence should end with a period. | 187 | |
Method 'getTotalEdges' is not designed for extension - needs to be abstract, final or empty. | 189 | |
First sentence should end with a period. | 193 | |
Method 'getTotalVertices' is not designed for extension - needs to be abstract, final or empty. | 195 |
Violation | Message | Line |
---|---|---|
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 47 | |
First sentence should end with a period. | 50 | |
First sentence should end with a period. | 52 | |
First sentence should end with a period. | 54 | |
First sentence should end with a period. | 56 | |
First sentence should end with a period. | 58 | |
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 63 | |
First sentence should end with a period. | 65 | |
First sentence should end with a period. | 67 | |
Parameter conf should be final. | 81 | |
'conf' hides a field. | 81 | |
Method 'validateClassTypes' is not designed for extension - needs to be abstract, final or empty. | 91 | |
'+' should be on a new line. | 117 | |
'+' should be on a new line. | 118 | |
'+' should be on a new line. | 125 | |
'+' should be on a new line. | 126 | |
'+' should be on a new line. | 133 | |
'+' should be on a new line. | 134 | |
'+' should be on a new line. | 149 | |
'+' should be on a new line. | 150 | |
'+' should be on a new line. | 155 | |
'+' should be on a new line. | 156 | |
'+' should be on a new line. | 175 | |
'+' should be on a new line. | 176 | |
'+' should be on a new line. | 183 | |
'+' should be on a new line. | 184 | |
'+' should be on a new line. | 191 | |
'+' should be on a new line. | 192 | |
'&&' should be on a new line. | 207 | |
'+' should be on a new line. | 210 | |
'+' should be on a new line. | 211 | |
'&&' should be on a new line. | 214 | |
'+' should be on a new line. | 217 | |
'+' should be on a new line. | 218 | |
'&&' should be on a new line. | 221 | |
'+' should be on a new line. | 224 | |
'+' should be on a new line. | 225 | |
'&&' should be on a new line. | 228 | |
'+' should be on a new line. | 231 | |
'+' should be on a new line. | 232 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 32 | |
First sentence should end with a period. | 34 | |
First sentence should end with a period. | 36 | |
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 40 | |
Method 'addPartitionStats' is not designed for extension - needs to be abstract, final or empty. | 48 | |
Parameter partitionStats should be final. | 48 | |
Method 'getVertexCount' is not designed for extension - needs to be abstract, final or empty. | 54 | |
Missing a Javadoc comment. | 54 | |
Method 'getFinishedVertexCount' is not designed for extension - needs to be abstract, final or empty. | 58 | |
Missing a Javadoc comment. | 58 | |
Method 'getEdgeCount' is not designed for extension - needs to be abstract, final or empty. | 62 | |
Missing a Javadoc comment. | 62 | |
Method 'getMessageCount' is not designed for extension - needs to be abstract, final or empty. | 66 | |
Missing a Javadoc comment. | 66 | |
Method 'getHaltComputation' is not designed for extension - needs to be abstract, final or empty. | 70 | |
Missing a Javadoc comment. | 70 | |
Method 'setHaltComputation' is not designed for extension - needs to be abstract, final or empty. | 74 | |
Missing a Javadoc comment. | 74 | |
Parameter value should be final. | 74 | |
Method 'addMessageCount' is not designed for extension - needs to be abstract, final or empty. | 83 | |
Parameter messageCount should be final. | 83 | |
'messageCount' hides a field. | 83 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 87 | |
Parameter input should be final. | 88 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 96 | |
Parameter output should be final. | 97 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 105 | |
'+' should be on a new line. | 107 | |
'+' should be on a new line. | 108 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 64 | |
First sentence should end with a period. | 66 | |
First sentence should end with a period. | 68 | |
First sentence should end with a period. | 70 | |
First sentence should end with a period. | 74 | |
First sentence should end with a period. | 76 | |
First sentence should end with a period. | 88 | |
First sentence should end with a period. | 90 | |
First sentence should end with a period. | 92 | |
First sentence should end with a period. | 94 | |
First sentence should end with a period. | 96 | |
First sentence should end with a period. | 98 | |
First sentence should end with a period. | 100 | |
Method 'getMapFunctions' is not designed for extension - needs to be abstract, final or empty. | 109 | |
First sentence should end with a period. | 113 | |
First sentence should end with a period. | 122 | |
Missing a Javadoc comment. | 131 | |
Missing a Javadoc comment. | 135 | |
Parameter t should be final. | 144 | |
Parameter e should be final. | 144 | |
'+' should be on a new line. | 146 | |
Method 'determineClassTypes' is not designed for extension - needs to be abstract, final or empty. | 157 | |
Parameter conf should be final. | 157 | |
'conf' hides a field. | 157 | |
'3' is a magic number. | 165 | |
Parameter myClass should be final. | 188 | |
Parameter conf should be final. | 225 | |
Parameter zkManager should be final. | 226 | |
Method 'setup' is not designed for extension - needs to be abstract, final or empty. | 262 | |
Parameter context should be final. | 263 | |
'+' should be on a new line. | 323 | |
'*' should be on a new line. | 330 | |
'||' should be on a new line. | 337 | |
'||' should be on a new line. | 338 | |
'||' should be on a new line. | 339 | |
'+' should be on a new line. | 342 | |
'||' should be on a new line. | 353 | |
'||' should be on a new line. | 354 | |
Method 'map' is not designed for extension - needs to be abstract, final or empty. | 382 | |
Method length is 152 lines (max allowed is 150). | 382 | |
Parameter key should be final. | 383 | |
Parameter value should be final. | 383 | |
Parameter context should be final. | 383 | |
'||' should be on a new line. | 398 | |
'+' should be on a new line. | 407 | |
'15000' is a magic number. | 477 | |
':' should be on a new line. | 479 | |
'||' should be on a new line. | 494 | |
'+' should be on a new line. | 523 | |
'+' should be on a new line. | 524 | |
'+' should be on a new line. | 529 | |
Method 'cleanup' is not designed for extension - needs to be abstract, final or empty. | 537 | |
Parameter context should be final. | 538 | |
Method 'run' is not designed for extension - needs to be abstract, final or empty. | 564 | |
Parameter context should be final. | 565 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 37 | |
First sentence should end with a period. | 39 | |
First sentence should end with a period. | 41 | |
First sentence should end with a period. | 43 | |
First sentence should end with a period. | 45 | |
First sentence should end with a period. | 47 | |
Method 'getSuperstep' is not designed for extension - needs to be abstract, final or empty. | 50 | |
Missing a Javadoc comment. | 50 | |
Method 'setSuperstep' is not designed for extension - needs to be abstract, final or empty. | 60 | |
Parameter superstep should be final. | 60 | |
'superstep' hides a field. | 60 | |
Method 'getTotalNumVertices' is not designed for extension - needs to be abstract, final or empty. | 65 | |
Missing a Javadoc comment. | 65 | |
Method 'setTotalNumVertices' is not designed for extension - needs to be abstract, final or empty. | 75 | |
Parameter numVertices should be final. | 75 | |
'numVertices' hides a field. | 75 | |
Method 'getTotalNumEdges' is not designed for extension - needs to be abstract, final or empty. | 80 | |
Missing a Javadoc comment. | 80 | |
Method 'setTotalNumEdges' is not designed for extension - needs to be abstract, final or empty. | 90 | |
Parameter numEdges should be final. | 90 | |
'numEdges' hides a field. | 90 | |
Method 'getContext' is not designed for extension - needs to be abstract, final or empty. | 95 | |
Missing a Javadoc comment. | 95 | |
Method 'setContext' is not designed for extension - needs to be abstract, final or empty. | 105 | |
Parameter context should be final. | 105 | |
'context' hides a field. | 105 | |
Method 'getGraphMapper' is not designed for extension - needs to be abstract, final or empty. | 110 | |
Missing a Javadoc comment. | 110 | |
Method 'setGraphMapper' is not designed for extension - needs to be abstract, final or empty. | 120 | |
Parameter graphMapper should be final. | 121 | |
'graphMapper' hides a field. | 121 | |
Method 'setWorkerCommunications' is not designed for extension - needs to be abstract, final or empty. | 132 | |
Parameter workerCommunications should be final. | 133 | |
'workerCommunications' hides a field. | 133 | |
Method 'getWorkerCommunications' is not designed for extension - needs to be abstract, final or empty. | 138 | |
Missing a Javadoc comment. | 138 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 57 | |
Variable 'edgeMap' must be private and have accessor methods. | 58 | |
First sentence should end with a period. | 59 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 62 | |
Parameter id should be final. | 64 | |
Parameter value should be final. | 64 | |
Parameter edges should be final. | 64 | |
Parameter messages should be final. | 64 | |
Parameter targetVertexId should be final. | 73 | |
Parameter value should be final. | 73 | |
'+' should be on a new line. | 76 | |
'+' should be on a new line. | 77 | |
Method 'hasEdge' is not designed for extension - needs to be abstract, final or empty. | 86 | |
Parameter targetVertexId should be final. | 87 | |
Method 'getEdges' is not designed for extension - needs to be abstract, final or empty. | 97 | |
Parameter edge should be final. | 103 | |
Method 'getEdgeValue' is not designed for extension - needs to be abstract, final or empty. | 109 | |
Parameter targetVertexId should be final. | 110 | |
Method 'getNumEdges' is not designed for extension - needs to be abstract, final or empty. | 114 | |
Method 'removeEdge' is not designed for extension - needs to be abstract, final or empty. | 119 | |
Parameter targetVertexId should be final. | 120 | |
Parameter message should be final. | 125 | |
Method 'putMessages' is not designed for extension - needs to be abstract, final or empty. | 131 | |
Parameter messages should be final. | 132 | |
Method 'getMessages' is not designed for extension - needs to be abstract, final or empty. | 137 | |
Method 'getNumMessages' is not designed for extension - needs to be abstract, final or empty. | 142 | |
Parameter in should be final. | 148 | |
Parameter out should be final. | 182 | |
Method 'releaseResources' is not designed for extension - needs to be abstract, final or empty. | 200 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 40 | |
First sentence should end with a period. | 42 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 45 | |
Parameter id should be final. | 46 | |
Parameter value should be final. | 46 | |
Parameter edges should be final. | 47 | |
Parameter messages should be final. | 48 | |
'messages' hides a field. | 48 | |
Avoid inline conditionals. | 50 | |
Avoid inline conditionals. | 57 | |
Method 'getNeighbors' is not designed for extension - needs to be abstract, final or empty. | 66 | |
Method 'hasEdge' is not designed for extension - needs to be abstract, final or empty. | 76 | |
Parameter targetVertexId should be final. | 77 | |
Method 'getNumEdges' is not designed for extension - needs to be abstract, final or empty. | 86 | |
Method 'getMessages' is not designed for extension - needs to be abstract, final or empty. | 91 | |
Method 'putMessages' is not designed for extension - needs to be abstract, final or empty. | 101 | |
Parameter newMessages should be final. | 102 | |
Method 'getNumMessages' is not designed for extension - needs to be abstract, final or empty. | 110 | |
Method 'releaseResources' is not designed for extension - needs to be abstract, final or empty. | 115 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 120 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 135 | |
Parameter in should be final. | 136 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 49 | |
First sentence should end with a period. | 51 | |
First sentence should end with a period. | 53 | |
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 58 | |
'zooKeeper' hides a field. | 65 | |
Parameter pathList should be final. | 66 | |
'pathList' hides a field. | 66 | |
'hostName' hides a field. | 66 | |
'19' is a magic number. | 106 | |
Avoid inline conditionals. | 108 | |
Parameter zkSplitPath should be final. | 120 | |
Expected @throws tag for 'IOException'. | 121 | |
Expected @throws tag for 'KeeperException'. | 121 | |
Expected @throws tag for 'InterruptedException'. | 121 | |
First sentence should end with a period. | 129 | |
Method 'iterator' is not designed for extension - needs to be abstract, final or empty. | 133 | |
First sentence should end with a period. | 141 | |
Method 'hasNext' is not designed for extension - needs to be abstract, final or empty. | 148 | |
First sentence should end with a period. | 153 | |
Method 'next' is not designed for extension - needs to be abstract, final or empty. | 156 | |
Method 'remove' is not designed for extension - needs to be abstract, final or empty. | 162 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 37 | |
First sentence should end with a period. | 44 | |
First sentence should end with a period. | 47 | |
First sentence should end with a period. | 50 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 53 | |
Parameter id should be final. | 54 | |
Parameter value should be final. | 54 | |
Parameter edges should be final. | 55 | |
Parameter messages should be final. | 56 | |
':' should be on a new line. | 59 | |
Parameter targetId should be final. | 73 | |
Parameter edgeValue should be final. | 74 | |
'+' should be on a new line. | 77 | |
'+' should be on a new line. | 78 | |
Method 'removeEdge' is not designed for extension - needs to be abstract, final or empty. | 87 | |
Parameter targetVertexId should be final. | 88 | |
Method 'getEdges' is not designed for extension - needs to be abstract, final or empty. | 99 | |
Method 'hasEdge' is not designed for extension - needs to be abstract, final or empty. | 133 | |
Parameter targetVertexId should be final. | 134 | |
Method 'getNumEdges' is not designed for extension - needs to be abstract, final or empty. | 138 | |
Parameter in should be final. | 144 | |
Parameter destVertexId should be final. | 173 | |
Parameter edgeValue should be final. | 173 | |
Parameter message should be final. | 187 | |
Method 'putMessages' is not designed for extension - needs to be abstract, final or empty. | 200 | |
Parameter messages should be final. | 201 | |
Method 'releaseResources' is not designed for extension - needs to be abstract, final or empty. | 208 | |
Method 'getNumMessages' is not designed for extension - needs to be abstract, final or empty. | 214 | |
Method 'getMessages' is not designed for extension - needs to be abstract, final or empty. | 219 | |
First sentence should end with a period. | 229 | |
Parameter elementList should be final. | 238 | |
'elementList' hides a field. | 238 | |
First sentence should end with a period. | 254 | |
First sentence should end with a period. | 256 | |
Parameter elementList should be final. | 264 | |
'elementList' hides a field. | 264 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 23 | |
Expected @throws tag for 'InstantiationException'. | 38 | |
Expected @throws tag for 'IllegalAccessException'. | 38 | |
Expected @throws tag for 'InstantiationException'. | 52 | |
Expected @throws tag for 'IllegalAccessException'. | 52 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 47 | |
First sentence should end with a period. | 49 | |
Expected @throws tag for 'InstantiationException'. | 61 | |
Expected @throws tag for 'IllegalAccessException'. | 62 | |
Method 'getSuperstep' is not designed for extension - needs to be abstract, final or empty. | 69 | |
Method 'getTotalNumVertices' is not designed for extension - needs to be abstract, final or empty. | 79 | |
Method 'getTotalNumEdges' is not designed for extension - needs to be abstract, final or empty. | 89 | |
Method 'haltComputation' is not designed for extension - needs to be abstract, final or empty. | 97 | |
Method 'isHalted' is not designed for extension - needs to be abstract, final or empty. | 106 | |
Method 'getGraphState' is not designed for extension - needs to be abstract, final or empty. | 115 | |
First sentence should end with a period. | 119 | |
Method 'setGraphState' is not designed for extension - needs to be abstract, final or empty. | 124 | |
Parameter graphState should be final. | 124 | |
'graphState' hides a field. | 124 | |
First sentence should end with a period. | 128 | |
Method 'getContext' is not designed for extension - needs to be abstract, final or empty. | 133 | |
Parameter name should be final. | 139 | |
Parameter aggregatorClass should be final. | 139 | |
Parameter name should be final. | 147 | |
Parameter aggregatorClass should be final. | 148 | |
Method 'getAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 154 | |
Parameter name should be final. | 155 | |
Method 'setAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 160 | |
Parameter name should be final. | 161 | |
Parameter value should be final. | 161 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 166 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 171 | |
Parameter conf should be final. | 172 | |
'conf' hides a field. | 172 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 48 | |
First sentence should end with a period. | 50 | |
First sentence should end with a period. | 52 | |
First sentence should end with a period. | 54 | |
First sentence should end with a period. | 58 | |
First sentence should end with a period. | 60 | |
Parameter bspServiceMaster should be final. | 71 | |
'bspServiceMaster' hides a field. | 71 | |
Parameter context should be final. | 72 | |
'context' hides a field. | 72 | |
Method 'run' is not designed for extension - needs to be abstract, final or empty. | 86 | |
'!=' should be on a new line. | 98 | |
'||' should be on a new line. | 99 | |
'1000.0d' is a magic number. | 105 | |
'-' should be on a new line. | 112 | |
'1000.0d' is a magic number. | 115 | |
'+' should be on a new line. | 117 | |
'+' should be on a new line. | 118 | |
'1000.0d' is a magic number. | 119 | |
'+' should be on a new line. | 119 | |
'+' should be on a new line. | 120 | |
'+' should be on a new line. | 121 | |
'+' should be on a new line. | 132 | |
'==' should be on a new line. | 158 | |
'+' should be on a new line. | 160 | |
'+' should be on a new line. | 163 | |
'+' should be on a new line. | 169 | |
'/' should be on a new line. | 170 | |
'1000.0d' is a magic number. | 171 | |
'+' should be on a new line. | 172 | |
'/' should be on a new line. | 173 | |
'1000.0d' is a magic number. | 174 | |
'+' should be on a new line. | 182 | |
'+' should be on a new line. | 186 | |
'+' should be on a new line. | 190 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 40 | |
Method 'instantiateVertex' is not designed for extension - needs to be abstract, final or empty. | 68 | |
Parameter vertexId should be final. | 69 | |
Parameter vertexValue should be final. | 69 | |
Parameter edges should be final. | 69 | |
Parameter messages should be final. | 69 | |
Method 'addVertexRequest' is not designed for extension - needs to be abstract, final or empty. | 84 | |
Parameter vertex should be final. | 84 | |
Expected @throws tag for 'IOException'. | 85 | |
Method 'removeVertexRequest' is not designed for extension - needs to be abstract, final or empty. | 96 | |
Parameter vertexId should be final. | 96 | |
Expected @throws tag for 'IOException'. | 96 | |
First sentence should end with a period. | 101 | |
Method 'addEdgeRequest' is not designed for extension - needs to be abstract, final or empty. | 108 | |
Parameter sourceVertexId should be final. | 108 | |
Parameter edge should be final. | 108 | |
Expected @throws tag for 'IOException'. | 109 | |
Method 'removeEdgeRequest' is not designed for extension - needs to be abstract, final or empty. | 121 | |
Parameter sourceVertexId should be final. | 121 | |
Parameter targetVertexId should be final. | 121 | |
Expected @throws tag for 'IOException'. | 122 |
Violation | Message | Line |
---|---|---|
Method 'getEdges' is not designed for extension - needs to be abstract, final or empty. | 53 | |
Parameter targetVertexId should be final. | 57 | |
Method 'getEdgeValue' is not designed for extension - needs to be abstract, final or empty. | 63 | |
Parameter targetVertexId should be final. | 64 | |
First sentence should end with a period. | 68 | |
Method 'addEdge' is not designed for extension - needs to be abstract, final or empty. | 76 | |
Parameter targetVertexId should be final. | 77 | |
Parameter value should be final. | 77 | |
First sentence should end with a period. | 81 | |
Method 'addEdgeRequest' is not designed for extension - needs to be abstract, final or empty. | 87 | |
Parameter sourceVertexId should be final. | 87 | |
Expected @throws tag for 'IOException'. | 87 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 93 | |
Parameter in should be final. | 94 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 125 | |
Parameter out should be final. | 126 |
Violation | Message | Line |
---|---|---|
Method 'getEdges' is not designed for extension - needs to be abstract, final or empty. | 55 | |
Parameter targetVertexId should be final. | 61 | |
Method 'getEdgeValue' is not designed for extension - needs to be abstract, final or empty. | 67 | |
Parameter targetVertexId should be final. | 68 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 72 | |
Parameter in should be final. | 73 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 104 | |
Parameter out should be final. | 105 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 41 | |
First sentence should end with a period. | 43 | |
First sentence should end with a period. | 45 | |
First sentence should end with a period. | 54 | |
First sentence should end with a period. | 56 | |
Variable 'output' must be private and have accessor methods. | 57 | |
First sentence should end with a period. | 58 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 61 | |
Parameter context should be final. | 63 | |
Parameter attempt should be final. | 63 | |
'+' should be on a new line. | 71 | |
Method 'writeAggregator' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Parameter aggregatorMap should be final. | 80 | |
Parameter superstep should be final. | 81 | |
Method 'aggregatorToString' is not designed for extension - needs to be abstract, final or empty. | 101 | |
Parameter aggregatorName should be final. | 101 | |
Parameter value should be final. | 102 | |
Parameter superstep should be final. | 103 | |
Parameter superstep should be final. | 115 | |
'||' should be on a new line. | 116 | |
Method 'close' is not designed for extension - needs to be abstract, final or empty. | 120 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 57 | |
'id' hides a field. | 71 | |
'value' hides a field. | 71 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 80 | |
Parameter id should be final. | 80 | |
'id' hides a field. | 80 | |
Parameter value should be final. | 80 | |
'value' hides a field. | 80 | |
Expected @throws tag for 'IOException'. | 92 | |
Method 'getSuperstep' is not designed for extension - needs to be abstract, final or empty. | 99 | |
Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 108 | |
First sentence should end with a period. | 112 | |
Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 117 | |
First sentence should end with a period. | 121 | |
Method 'setValue' is not designed for extension - needs to be abstract, final or empty. | 126 | |
Parameter value should be final. | 126 | |
'value' hides a field. | 126 | |
Method 'getTotalNumVertices' is not designed for extension - needs to be abstract, final or empty. | 136 | |
Method 'getTotalNumEdges' is not designed for extension - needs to be abstract, final or empty. | 146 | |
Method 'hasEdge' is not designed for extension - needs to be abstract, final or empty. | 163 | |
Parameter targetVertexId should be final. | 163 | |
Method 'getEdgeValue' is not designed for extension - needs to be abstract, final or empty. | 180 | |
Parameter targetVertexId should be final. | 180 | |
Method 'getNumEdges' is not designed for extension - needs to be abstract, final or empty. | 194 | |
Method 'sendMessage' is not designed for extension - needs to be abstract, final or empty. | 206 | |
Parameter id should be final. | 206 | |
'id' hides a field. | 206 | |
Parameter message should be final. | 206 | |
Method 'sendMessageToAllEdges' is not designed for extension - needs to be abstract, final or empty. | 220 | |
Parameter message should be final. | 220 | |
Method 'voteToHalt' is not designed for extension - needs to be abstract, final or empty. | 232 | |
Method 'wakeUp' is not designed for extension - needs to be abstract, final or empty. | 239 | |
Method 'isHalted' is not designed for extension - needs to be abstract, final or empty. | 248 | |
Method 'getNumMessages' is not designed for extension - needs to be abstract, final or empty. | 264 | |
First sentence should end with a period. | 268 | |
First sentence should end with a period. | 275 | |
Method 'getGraphState' is not designed for extension - needs to be abstract, final or empty. | 286 | |
First sentence should end with a period. | 290 | |
Method 'setGraphState' is not designed for extension - needs to be abstract, final or empty. | 295 | |
Parameter graphState should be final. | 295 | |
'graphState' hides a field. | 295 | |
First sentence should end with a period. | 299 | |
Method 'getContext' is not designed for extension - needs to be abstract, final or empty. | 304 | |
First sentence should end with a period. | 308 | |
Method 'getWorkerContext' is not designed for extension - needs to be abstract, final or empty. | 313 | |
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 317 | |
Parameter name should be final. | 318 | |
Parameter value should be final. | 318 | |
'value' hides a field. | 318 | |
Method 'getAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 323 | |
Parameter name should be final. | 324 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 329 | |
Parameter in should be final. | 330 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 358 | |
Parameter out should be final. | 359 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 377 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 382 | |
Parameter conf should be final. | 383 | |
'conf' hides a field. | 383 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 387 | |
'+' should be on a new line. | 389 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 54 |
Violation | Message | Line |
---|---|---|
Expected @throws tag for 'IOException'. | 46 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 25 | |
First sentence should end with a period. | 27 | |
Parameter vertexCount should be final. | 44 | |
'vertexCount' hides a field. | 44 | |
Parameter edgeCount should be final. | 44 | |
'edgeCount' hides a field. | 44 | |
Method 'getVertexCount' is not designed for extension - needs to be abstract, final or empty. | 49 | |
Missing a Javadoc comment. | 49 | |
Method 'getEdgeCount' is not designed for extension - needs to be abstract, final or empty. | 53 | |
Missing a Javadoc comment. | 53 | |
Method 'incrVertexEdgeCount' is not designed for extension - needs to be abstract, final or empty. | 63 | |
Parameter vertexEdgeCount should be final. | 64 | |
Method 'incrVertexEdgeCount' is not designed for extension - needs to be abstract, final or empty. | 77 | |
Parameter vertexCount should be final. | 78 | |
'vertexCount' hides a field. | 78 | |
Parameter edgeCount should be final. | 78 | |
'edgeCount' hides a field. | 78 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 84 |
Violation | Message | Line |
---|---|---|
Expected @throws tag for 'IOException'. | 64 | |
Expected @throws tag for 'InterruptedException'. | 64 | |
Expected @throws tag for 'IOException'. | 79 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 48 | |
First sentence should end with a period. | 51 | |
First sentence should end with a period. | 53 | |
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 57 | |
Method 'copy' is not designed for extension - needs to be abstract, final or empty. | 65 | |
Method 'getAddedVertexList' is not designed for extension - needs to be abstract, final or empty. | 75 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 80 | |
Parameter input should be final. | 81 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 109 | |
Parameter output should be final. | 110 | |
First sentence should end with a period. | 127 | |
Method 'addVertex' is not designed for extension - needs to be abstract, final or empty. | 132 | |
Parameter vertex should be final. | 132 | |
Method 'getRemovedVertexCount' is not designed for extension - needs to be abstract, final or empty. | 136 | |
First sentence should end with a period. | 141 | |
Method 'removeVertex' is not designed for extension - needs to be abstract, final or empty. | 144 | |
Method 'getAddedEdgeList' is not designed for extension - needs to be abstract, final or empty. | 148 | |
First sentence should end with a period. | 153 | |
Method 'addEdge' is not designed for extension - needs to be abstract, final or empty. | 158 | |
Parameter edge should be final. | 158 | |
Method 'getRemovedEdgeList' is not designed for extension - needs to be abstract, final or empty. | 162 | |
First sentence should end with a period. | 167 | |
Method 'removeEdge' is not designed for extension - needs to be abstract, final or empty. | 172 | |
Parameter destinationVertexId should be final. | 172 | |
First sentence should end with a period. | 176 | |
Method 'addVertexMutations' is not designed for extension - needs to be abstract, final or empty. | 181 | |
Parameter vertexMutations should be final. | 181 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 188 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 203 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 208 | |
Parameter conf should be final. | 209 | |
'conf' hides a field. | 209 |
Violation | Message | Line |
---|---|---|
Expected @throws tag for 'IOException'. | 52 | |
Expected @throws tag for 'InterruptedException'. | 52 | |
Expected @throws tag for 'InterruptedException'. | 67 | |
Expected @throws tag for 'IOException'. | 80 | |
Expected @throws tag for 'InterruptedException'. | 80 |
Violation | Message | Line |
---|---|---|
Expected @throws tag for 'IOException'. | 50 | |
Expected @throws tag for 'InterruptedException'. | 50 | |
Expected @throws tag for 'IOException'. | 58 | |
Expected @throws tag for 'InterruptedException'. | 58 | |
Expected @throws tag for 'IOException'. | 69 | |
Expected @throws tag for 'InterruptedException'. | 69 | |
Expected @throws tag for 'IOException'. | 76 | |
Expected @throws tag for 'IOException'. | 86 | |
Expected @throws tag for 'InterruptedException'. | 86 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 42 | |
First sentence should end with a period. | 44 | |
First sentence should end with a period. | 46 | |
Method 'resolve' is not designed for extension - needs to be abstract, final or empty. | 49 | |
Parameter vertexId should be final. | 51 | |
Parameter vertex should be final. | 52 | |
Parameter vertexChanges should be final. | 53 | |
Parameter hasMessages should be final. | 54 | |
'+' should be on a new line. | 69 | |
'+' should be on a new line. | 70 | |
'&&' should be on a new line. | 94 | |
'+' should be on a new line. | 96 | |
'+' should be on a new line. | 97 | |
'&&' should be on a new line. | 102 | |
Method 'instantiateVertex' is not designed for extension - needs to be abstract, final or empty. | 115 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 123 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 128 | |
Parameter conf should be final. | 129 | |
'conf' hides a field. | 129 | |
Method 'setGraphState' is not designed for extension - needs to be abstract, final or empty. | 138 | |
Parameter graphState should be final. | 138 | |
'graphState' hides a field. | 138 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 27 | |
Expected @throws tag for 'IOException'. | 44 | |
First sentence should end with a period. | 46 | |
Expected @throws tag for 'IOException'. | 54 | |
Expected @throws tag for 'InterruptedException'. | 54 | |
Expected @throws tag for 'IOException'. | 64 | |
Expected @throws tag for 'InterruptedException'. | 64 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 23 | |
First sentence should end with a period. | 27 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 30 | |
Method 'setGraphState' is not designed for extension - needs to be abstract, final or empty. | 38 | |
Parameter graphState should be final. | 38 | |
'graphState' hides a field. | 38 | |
Method 'getSuperstep' is not designed for extension - needs to be abstract, final or empty. | 79 | |
Method 'getTotalNumVertices' is not designed for extension - needs to be abstract, final or empty. | 89 | |
Method 'getTotalNumEdges' is not designed for extension - needs to be abstract, final or empty. | 99 | |
First sentence should end with a period. | 103 | |
Method 'getContext' is not designed for extension - needs to be abstract, final or empty. | 108 | |
Method 'aggregate' is not designed for extension - needs to be abstract, final or empty. | 112 | |
Parameter name should be final. | 113 | |
Parameter value should be final. | 113 | |
Method 'getAggregatedValue' is not designed for extension - needs to be abstract, final or empty. | 118 | |
Parameter name should be final. | 119 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 32 | |
First sentence should end with a period. | 34 | |
First sentence should end with a period. | 36 | |
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 41 | |
Parameter hostname should be final. | 54 | |
'hostname' hides a field. | 54 | |
Parameter partitionId should be final. | 54 | |
'partitionId' hides a field. | 54 | |
Parameter port should be final. | 54 | |
'port' hides a field. | 54 | |
Method 'getHostname' is not designed for extension - needs to be abstract, final or empty. | 61 | |
Missing a Javadoc comment. | 61 | |
Method 'getPartitionId' is not designed for extension - needs to be abstract, final or empty. | 65 | |
Missing a Javadoc comment. | 65 | |
Method 'getHostnameId' is not designed for extension - needs to be abstract, final or empty. | 69 | |
Missing a Javadoc comment. | 69 | |
First sentence should end with a period. | 73 | |
Method 'getInetSocketAddress' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Method 'getPort' is not designed for extension - needs to be abstract, final or empty. | 82 | |
Missing a Javadoc comment. | 82 | |
Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 86 | |
Parameter other should be final. | 87 | |
'&&' should be on a new line. | 90 | |
'&&' should be on a new line. | 91 | |
Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 99 | |
'17' is a magic number. | 101 | |
'37' is a magic number. | 102 | |
'37' is a magic number. | 103 | |
'37' is a magic number. | 104 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 108 | |
'+' should be on a new line. | 110 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 114 | |
Parameter input should be final. | 115 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 122 | |
Parameter output should be final. | 123 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 34 | |
First sentence should end with a period. | 36 | |
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 40 | |
First sentence should end with a period. | 42 | |
Parameter partitionId should be final. | 56 | |
'partitionId' hides a field. | 56 | |
Parameter workerInfo should be final. | 56 | |
'workerInfo' hides a field. | 56 | |
Parameter partitionId should be final. | 68 | |
'partitionId' hides a field. | 68 | |
Parameter workerInfo should be final. | 69 | |
'workerInfo' hides a field. | 69 | |
Parameter previousWorkerInfo should be final. | 70 | |
'previousWorkerInfo' hides a field. | 70 | |
Parameter checkpointFilesPrefix should be final. | 71 | |
'checkpointFilesPrefix' hides a field. | 71 | |
Method 'getPartitionId' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Method 'getWorkerInfo' is not designed for extension - needs to be abstract, final or empty. | 83 | |
Method 'setWorkerInfo' is not designed for extension - needs to be abstract, final or empty. | 88 | |
Parameter workerInfo should be final. | 89 | |
'workerInfo' hides a field. | 89 | |
Method 'getPreviousWorkerInfo' is not designed for extension - needs to be abstract, final or empty. | 93 | |
Method 'setPreviousWorkerInfo' is not designed for extension - needs to be abstract, final or empty. | 98 | |
Parameter workerInfo should be final. | 99 | |
'workerInfo' hides a field. | 99 | |
Method 'getCheckpointFilesPrefix' is not designed for extension - needs to be abstract, final or empty. | 103 | |
Method 'setCheckpointFilesPrefix' is not designed for extension - needs to be abstract, final or empty. | 108 | |
Parameter checkpointFilesPrefix should be final. | 109 | |
'checkpointFilesPrefix' hides a field. | 109 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 113 | |
Parameter input should be final. | 114 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 129 | |
Parameter output should be final. | 130 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 147 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 152 | |
Parameter conf should be final. | 153 | |
'conf' hides a field. | 153 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 157 | |
'+' should be on a new line. | 159 |
Violation | Message | Line |
---|---|---|
Parameter conf should be final. | 83 | |
'conf' hides a field. | 83 | |
'+' should be on a new line. | 89 | |
Parameter partitionId should be final. | 100 | |
Parameter partitionId should be final. | 111 | |
Parameter partitionId should be final. | 126 | |
Parameter partition should be final. | 136 | |
Expected @throws tag for 'IOException'. | 137 | |
Parameter partitionId should be final. | 156 | |
Expected @throws tag for 'IOException'. | 157 | |
Parameter partitionId should be final. | 181 | |
Parameter vertices should be final. | 182 | |
Expected @throws tag for 'IOException'. | 183 | |
Parameter partitionId should be final. | 198 | |
'+' should be on a new line. | 204 | |
'+' should be on a new line. | 213 | |
'+' should be on a new line. | 218 | |
'+' should be on a new line. | 224 | |
Parameter partition should be final. | 234 | |
'+' should be on a new line. | 246 | |
Method 'addPartition' is not designed for extension - needs to be abstract, final or empty. | 251 | |
Parameter partition should be final. | 252 | |
'+' should be on a new line. | 255 | |
Method 'addPartitionVertices' is not designed for extension - needs to be abstract, final or empty. | 262 | |
Parameter partitionId should be final. | 263 | |
Parameter vertices should be final. | 264 | |
'+' should be on a new line. | 279 | |
Method 'getPartition' is not designed for extension - needs to be abstract, final or empty. | 300 | |
Parameter partitionId should be final. | 301 | |
'+' should be on a new line. | 308 | |
Method 'removePartition' is not designed for extension - needs to be abstract, final or empty. | 313 | |
Parameter partitionId should be final. | 314 | |
'+' should be on a new line. | 325 | |
Method 'deletePartition' is not designed for extension - needs to be abstract, final or empty. | 336 | |
Parameter partitionId should be final. | 337 | |
Method 'hasPartition' is not designed for extension - needs to be abstract, final or empty. | 352 | |
Parameter partitionId should be final. | 353 | |
Method 'getPartitionIds' is not designed for extension - needs to be abstract, final or empty. | 357 | |
Method 'getNumPartitions' is not designed for extension - needs to be abstract, final or empty. | 363 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 44 | |
First sentence should end with a period. | 47 | |
First sentence should end with a period. | 49 | |
First sentence should end with a period. | 52 | |
First sentence should end with a period. | 54 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 55 | |
First sentence should end with a period. | 56 | |
'1024' is a magic number. | 60 | |
'1024' is a magic number. | 60 | |
'350' is a magic number. | 60 | |
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 63 | |
First sentence should end with a period. | 65 | |
First sentence should end with a period. | 67 | |
Parameter conf should be final. | 75 | |
'conf' hides a field. | 75 | |
Method 'createInitialPartitionOwners' is not designed for extension - needs to be abstract, final or empty. | 81 | |
Parameter availableWorkerInfos should be final. | 83 | |
Parameter maxWorkers should be final. | 83 | |
'*' should be on a new line. | 95 | |
'+' should be on a new line. | 102 | |
'+' should be on a new line. | 103 | |
'*' should be on a new line. | 104 | |
'+' should be on a new line. | 108 | |
'+' should be on a new line. | 109 | |
Method 'getCurrentPartitionOwners' is not designed for extension - needs to be abstract, final or empty. | 125 | |
Method 'setPartitionOwnerList' is not designed for extension - needs to be abstract, final or empty. | 135 | |
Parameter partitionOwnerList should be final. | 135 | |
'partitionOwnerList' hides a field. | 136 | |
Method 'generateChangedPartitionOwners' is not designed for extension - needs to be abstract, final or empty. | 140 | |
Parameter allPartitionStatsList should be final. | 142 | |
Parameter availableWorkerInfos should be final. | 143 | |
Parameter maxWorkers should be final. | 144 | |
Parameter superstep should be final. | 145 | |
Method 'createPartitionStats' is not designed for extension - needs to be abstract, final or empty. | 153 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 39 | |
Method 'createMasterGraphPartitioner' is not designed for extension - needs to be abstract, final or empty. | 42 | |
Method 'createWorkerGraphPartitioner' is not designed for extension - needs to be abstract, final or empty. | 47 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 52 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Parameter conf should be final. | 58 | |
'conf' hides a field. | 58 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 39 | |
Method 'createMasterGraphPartitioner' is not designed for extension - needs to be abstract, final or empty. | 42 | |
Method 'createWorkerGraphPartitioner' is not designed for extension - needs to be abstract, final or empty. | 47 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 52 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Parameter conf should be final. | 58 | |
'conf' hides a field. | 58 |
Violation | Message | Line |
---|---|---|
Method 'getPartitionOwner' is not designed for extension - needs to be abstract, final or empty. | 36 | |
Parameter vertexId should be final. | 37 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 45 | |
Variable 'partitionOwnerList' must be private and have accessor methods. | 46 | |
Method 'createPartitionOwner' is not designed for extension - needs to be abstract, final or empty. | 49 | |
Method 'getPartitionOwner' is not designed for extension - needs to be abstract, final or empty. | 54 | |
Parameter vertexId should be final. | 55 | |
'%' should be on a new line. | 57 | |
Method 'finalizePartitionStats' is not designed for extension - needs to be abstract, final or empty. | 62 | |
Parameter workerPartitionStats should be final. | 64 | |
Parameter partitionStore should be final. | 65 | |
Method 'updatePartitionOwners' is not designed for extension - needs to be abstract, final or empty. | 70 | |
Parameter myWorkerInfo should be final. | 72 | |
Parameter masterSetPartitionOwners should be final. | 73 | |
Parameter partitionStore should be final. | 74 | |
'&&' should be on a new line. | 87 | |
'+' should be on a new line. | 91 | |
'+' should be on a new line. | 92 | |
Method 'getPartitionOwners' is not designed for extension - needs to be abstract, final or empty. | 116 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 76 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 50 | |
First sentence should end with a period. | 52 | |
First sentence should end with a period. | 54 | |
Parameter conf should be final. | 63 | |
'conf' hides a field. | 63 | |
Parameter id should be final. | 63 | |
'id' hides a field. | 63 | |
Method 'getVertex' is not designed for extension - needs to be abstract, final or empty. | 80 | |
Parameter vertexIndex should be final. | 80 | |
First sentence should end with a period. | 84 | |
Method 'putVertex' is not designed for extension - needs to be abstract, final or empty. | 90 | |
Parameter vertex should be final. | 90 | |
First sentence should end with a period. | 94 | |
Method 'removeVertex' is not designed for extension - needs to be abstract, final or empty. | 100 | |
Parameter vertexIndex should be final. | 100 | |
Method 'getVertices' is not designed for extension - needs to be abstract, final or empty. | 109 | |
Method 'putVertices' is not designed for extension - needs to be abstract, final or empty. | 118 | |
Parameter vertices should be final. | 118 | |
Method 'getEdgeCount' is not designed for extension - needs to be abstract, final or empty. | 129 | |
Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 142 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 146 | |
'+' should be on a new line. | 148 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 152 | |
Parameter input should be final. | 153 | |
'+' should be on a new line. | 162 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 168 | |
Parameter output should be final. | 169 |
Violation | Message | Line |
---|---|---|
Class PartitionBalancer should be declared as final. | 37 | |
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 41 | |
First sentence should end with a period. | 44 | |
First sentence should end with a period. | 47 | |
First sentence should end with a period. | 50 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 51 | |
First sentence should end with a period. | 57 | |
First sentence should end with a period. | 59 | |
First sentence should end with a period. | 61 | |
Parameter partitionStat should be final. | 77 | |
Parameter balanceValue should be final. | 78 | |
First sentence should end with a period. | 90 | |
First sentence should end with a period. | 95 | |
First sentence should end with a period. | 97 | |
Parameter ownerStatMap should be final. | 108 | |
'ownerStatMap' hides a field. | 108 | |
Parameter balanceValue should be final. | 109 | |
'balanceValue' hides a field. | 109 | |
Parameter owner1 should be final. | 115 | |
Parameter owner2 should be final. | 115 | |
'-' should be on a new line. | 117 | |
First sentence should end with a period. | 128 | |
First sentence should end with a period. | 130 | |
First sentence should end with a period. | 132 | |
First sentence should end with a period. | 134 | |
Parameter workerInfo should be final. | 145 | |
'workerInfo' hides a field. | 145 | |
Parameter balanceValue should be final. | 146 | |
'balanceValue' hides a field. | 146 | |
Parameter ownerStatsMap should be final. | 147 | |
'ownerStatsMap' hides a field. | 147 | |
Parameter partitionOwner should be final. | 168 | |
Parameter other should be final. | 181 | |
Parameter conf should be final. | 197 | |
Parameter partitionOwners should be final. | 198 | |
Parameter allPartitionStats should be final. | 199 | |
Parameter availableWorkerInfos should be final. | 200 | |
'+' should be on a new line. | 205 | |
'+' should be on a new line. | 217 | |
'!=' should be on a new line. | 225 | |
'+' should be on a new line. | 228 | |
'+' should be on a new line. | 239 | |
'+' should be on a new line. | 244 | |
'+' should be on a new line. | 250 | |
'+' should be on a new line. | 251 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 32 | |
First sentence should end with a period. | 34 | |
Parameter myDependencyWorkerSet should be final. | 44 | |
'myDependencyWorkerSet' hides a field. | 44 | |
Parameter sendWorkerPartitionMap should be final. | 45 | |
'sendWorkerPartitionMap' hides a field. | 45 | |
First sentence should end with a period. | 50 | |
Method 'getMyDependencyWorkerSet' is not designed for extension - needs to be abstract, final or empty. | 55 | |
Method 'getSendWorkerPartitionMap' is not designed for extension - needs to be abstract, final or empty. | 64 | |
Method 'doExchange' is not designed for extension - needs to be abstract, final or empty. | 73 | |
'||' should be on a new line. | 74 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 28 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 32 | |
First sentence should end with a period. | 34 | |
First sentence should end with a period. | 36 | |
First sentence should end with a period. | 38 | |
Parameter partitionId should be final. | 54 | |
'partitionId' hides a field. | 54 | |
Parameter vertexCount should be final. | 55 | |
'vertexCount' hides a field. | 55 | |
Parameter finishedVertexCount should be final. | 56 | |
'finishedVertexCount' hides a field. | 56 | |
Parameter edgeCount should be final. | 57 | |
'edgeCount' hides a field. | 57 | |
Method 'setPartitionId' is not designed for extension - needs to be abstract, final or empty. | 69 | |
Parameter partitionId should be final. | 69 | |
'partitionId' hides a field. | 69 | |
Method 'getPartitionId' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Method 'incrVertexCount' is not designed for extension - needs to be abstract, final or empty. | 85 | |
Method 'getVertexCount' is not designed for extension - needs to be abstract, final or empty. | 94 | |
Method 'incrFinishedVertexCount' is not designed for extension - needs to be abstract, final or empty. | 101 | |
Method 'getFinishedVertexCount' is not designed for extension - needs to be abstract, final or empty. | 110 | |
Method 'addEdgeCount' is not designed for extension - needs to be abstract, final or empty. | 119 | |
Parameter edgeCount should be final. | 119 | |
'edgeCount' hides a field. | 119 | |
Method 'getEdgeCount' is not designed for extension - needs to be abstract, final or empty. | 128 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 132 | |
Parameter input should be final. | 133 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 140 | |
Parameter output should be final. | 141 | |
Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 148 | |
'+' should be on a new line. | 150 |
Violation | Message | Line |
---|---|---|
Method 'isEmpty' is not designed for extension - needs to be abstract, final or empty. | 109 | |
Method 'getPartitions' is not designed for extension - needs to be abstract, final or empty. | 118 | |
Parameter partitionId should be final. | 122 |
Violation | Message | Line |
---|---|---|
Class PartitionUtils should be declared as final. | 39 | |
First sentence should end with a period. | 40 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 41 | |
Parameter worker1 should be final. | 54 | |
Parameter worker2 should be final. | 55 | |
'-' should be on a new line. | 56 | |
Parameter worker1 should be final. | 68 | |
Parameter worker2 should be final. | 69 | |
'-' should be on a new line. | 70 | |
Parameter partitionOwnerList should be final. | 83 | |
Parameter allPartitionStats should be final. | 84 | |
'+' should be on a new line. | 91 | |
'+' should be on a new line. | 126 | |
'/' should be on a new line. | 127 | |
'+' should be on a new line. | 128 | |
'+' should be on a new line. | 129 | |
'+' should be on a new line. | 130 | |
'+' should be on a new line. | 131 | |
'+' should be on a new line. | 132 | |
'+' should be on a new line. | 133 | |
'+' should be on a new line. | 134 | |
'+' should be on a new line. | 138 | |
'/' should be on a new line. | 139 | |
'+' should be on a new line. | 140 | |
'+' should be on a new line. | 141 | |
'+' should be on a new line. | 142 | |
'+' should be on a new line. | 143 | |
'+' should be on a new line. | 144 | |
'+' should be on a new line. | 145 | |
'+' should be on a new line. | 146 |
Violation | Message | Line |
---|---|---|
Method 'createPartitionStats' is not designed for extension - needs to be abstract, final or empty. | 39 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 37 | |
Parameter maxIndex should be final. | 50 | |
'maxIndex' hides a field. | 50 | |
Method 'getMaxIndex' is not designed for extension - needs to be abstract, final or empty. | 59 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 63 | |
Parameter input should be final. | 64 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 70 | |
Parameter output should be final. | 71 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 36 | |
First sentence should end with a period. | 39 | |
Method 'getRangeSplitHint' is not designed for extension - needs to be abstract, final or empty. | 44 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 48 | |
Parameter input should be final. | 49 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 60 | |
Parameter output should be final. | 61 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 40 | |
First sentence should end with a period. | 42 | |
First sentence should end with a period. | 44 | |
First sentence should end with a period. | 46 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 49 | |
Parameter input should be final. | 50 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Parameter output should be final. | 58 | |
Method 'getConf' is not designed for extension - needs to be abstract, final or empty. | 64 | |
Method 'setConf' is not designed for extension - needs to be abstract, final or empty. | 69 | |
Parameter conf should be final. | 70 | |
'conf' hides a field. | 70 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 48 | |
Variable 'vertexRangeMap' must be private and have accessor methods. | 49 | |
Method 'createPartitionOwner' is not designed for extension - needs to be abstract, final or empty. | 52 | |
Method 'getPartitionOwner' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Parameter vertexId should be final. | 58 | |
Method 'getPartitionOwners' is not designed for extension - needs to be abstract, final or empty. | 74 |
Violation | Message | Line |
---|---|---|
Parameter conf should be final. | 53 | |
'conf' hides a field. | 53 | |
Method 'addPartition' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Parameter partition should be final. | 58 | |
'+' should be on a new line. | 60 | |
Method 'addPartitionVertices' is not designed for extension - needs to be abstract, final or empty. | 65 | |
Parameter partitionId should be final. | 66 | |
Parameter vertices should be final. | 67 | |
Method 'getPartition' is not designed for extension - needs to be abstract, final or empty. | 80 | |
Parameter partitionId should be final. | 81 | |
Method 'removePartition' is not designed for extension - needs to be abstract, final or empty. | 85 | |
Parameter partitionId should be final. | 86 | |
Method 'deletePartition' is not designed for extension - needs to be abstract, final or empty. | 90 | |
Parameter partitionId should be final. | 91 | |
Method 'hasPartition' is not designed for extension - needs to be abstract, final or empty. | 95 | |
Parameter partitionId should be final. | 96 | |
Method 'getPartitionIds' is not designed for extension - needs to be abstract, final or empty. | 100 | |
Method 'getNumPartitions' is not designed for extension - needs to be abstract, final or empty. | 105 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 48 |
Violation | Message | Line |
---|---|---|
Method 'getServices' is not designed for extension - needs to be abstract, final or empty. | 38 |
Violation | Message | Line |
---|---|---|
Method 'selectToken' is not designed for extension - needs to be abstract, final or empty. | 34 | |
Parameter service should be final. | 36 | |
Parameter tokens should be final. | 37 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 64 | |
Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 65 | |
Parameter conf should be final. | 73 | |
Parameter allPartitionStatsList should be final. | 79 | |
Parameter availableWorkerInfos should be final. | 80 | |
Parameter maxWorkers should be final. | 81 | |
Parameter superstep should be final. | 82 | |
'+' should be on a new line. | 97 | |
'+' should be on a new line. | 98 | |
':' should be on a new line. | 103 | |
'?' should be on a new line. | 106 | |
Avoid inline conditionals. | 106 | |
'+' should be on a new line. | 114 | |
Method 'createMasterGraphPartitioner' is not designed for extension - needs to be abstract, final or empty. | 122 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 57 | |
First sentence should end with a period. | 59 | |
Parameter recordWriter should be final. | 67 | |
Parameter vertex should be final. | 72 | |
Method 'createVertexWriter' is not designed for extension - needs to be abstract, final or empty. | 92 | |
Parameter context should be final. | 93 | |
'(' should be on the previous line. | 96 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 53 | |
First sentence should end with a period. | 55 | |
First sentence should end with a period. | 57 | |
Parameter lineRecordReader should be final. | 84 | |
Parameter lineRecordReader should be final. | 96 | |
Parameter sanitizer should be final. | 97 | |
'sanitizer' hides a field. | 97 | |
First sentence should end with a period. | 117 | |
Method 'nextVertex' is not designed for extension - needs to be abstract, final or empty. | 126 | |
Method 'getCurrentVertex' is not designed for extension - needs to be abstract, final or empty. | 131 |
Violation | Message | Line |
---|---|---|
Method 'getSplits' is not designed for extension - needs to be abstract, final or empty. | 45 | |
Parameter context should be final. | 46 | |
Parameter numWorkers should be final. | 46 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 56 | |
First sentence should end with a period. | 58 | |
First sentence should end with a period. | 64 | |
Parameter recordWriter should be final. | 72 | |
Parameter vertex should be final. | 77 | |
Method 'createVertexWriter' is not designed for extension - needs to be abstract, final or empty. | 103 | |
Parameter context should be final. | 104 | |
'(' should be on the previous line. | 107 |
Violation | Message | Line |
---|---|---|
Method 'createVertexReader' is not designed for extension - needs to be abstract, final or empty. | 49 | |
Parameter split should be final. | 51 | |
Parameter context should be final. | 51 | |
First sentence should end with a period. | 63 | |
Parameter lineReader should be final. | 71 | |
Method 'getCurrentVertex' is not designed for extension - needs to be abstract, final or empty. | 76 | |
Method 'nextVertex' is not designed for extension - needs to be abstract, final or empty. | 99 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 21 | |
Class JsonBase64VertexFormat should be declared as final. | 24 | |
First sentence should end with a period. | 25 | |
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 29 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 61 | |
Parameter lineRecordReader should be final. | 78 | |
Method 'createVertexReader' is not designed for extension - needs to be abstract, final or empty. | 158 | |
Parameter split should be final. | 160 | |
Parameter context should be final. | 160 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 54 | |
Parameter lineRecordWriter should be final. | 70 | |
Parameter vertex should be final. | 75 | |
Method 'createVertexWriter' is not designed for extension - needs to be abstract, final or empty. | 119 | |
Parameter context should be final. | 120 |
Violation | Message | Line |
---|---|---|
Method 'createVertexReader' is not designed for extension - needs to be abstract, final or empty. | 48 | |
Parameter split should be final. | 50 | |
Parameter context should be final. | 51 | |
Parameter lineRecordReader should be final. | 77 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 35 | |
Method 'createVertexWriter' is not designed for extension - needs to be abstract, final or empty. | 42 | |
Parameter context should be final. | 44 | |
Parameter lineRecordWriter should be final. | 64 | |
Parameter vertex should be final. | 69 |
Violation | Message | Line |
---|---|---|
Parameter lineRecordReader should be final. | 60 | |
Parameter lineRecordReader should be final. | 70 | |
Parameter sanitizer should be final. | 71 | |
Parameter s should be final. | 76 | |
Parameter id should be final. | 76 | |
Parameter s should be final. | 81 | |
Parameter value should be final. | 81 | |
Parameter s1 should be final. | 87 | |
Parameter s2 should be final. | 88 | |
Parameter textIntWritableEdge should be final. | 89 | |
Method 'createVertexReader' is not designed for extension - needs to be abstract, final or empty. | 95 | |
Parameter split should be final. | 98 | |
Parameter context should be final. | 99 |
Violation | Message | Line |
---|---|---|
Method 'createVertexReader' is not designed for extension - needs to be abstract, final or empty. | 72 | |
Parameter split should be final. | 74 | |
Parameter context should be final. | 74 | |
First sentence should end with a period. | 100 | |
Parameter inputSplit should be final. | 110 | |
Parameter context should be final. | 111 | |
'*' should be on a new line. | 129 | |
'+' should be on a new line. | 130 | |
'+' should be on a new line. | 135 | |
'%' should be on a new line. | 168 | |
'+' should be on a new line. | 176 | |
'+' should be on a new line. | 177 | |
'+' should be on a new line. | 178 | |
'100.0f' is a magic number. | 190 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 48 | |
Variable 'sequenceFileInputFormat' must be private and have accessor methods. | 49 | |
Method 'getSplits' is not designed for extension - needs to be abstract, final or empty. | 52 | |
Parameter context should be final. | 53 | |
Parameter numWorkers should be final. | 53 | |
Method 'createVertexReader' is not designed for extension - needs to be abstract, final or empty. | 58 | |
Parameter split should be final. | 59 | |
Parameter context should be final. | 60 | |
First sentence should end with a period. | 78 | |
Parameter recordReader should be final. | 86 | |
'recordReader' hides a field. | 86 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 90 | |
Parameter inputSplit should be final. | 90 | |
Parameter context should be final. | 91 | |
Method 'nextVertex' is not designed for extension - needs to be abstract, final or empty. | 95 | |
Method 'getCurrentVertex' is not designed for extension - needs to be abstract, final or empty. | 100 | |
Method 'close' is not designed for extension - needs to be abstract, final or empty. | 106 | |
Method 'getProgress' is not designed for extension - needs to be abstract, final or empty. | 110 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 42 | |
Parameter lineRecordReader should be final. | 55 | |
First sentence should end with a period. | 59 | |
Parameter lineRecordReader should be final. | 65 | |
Parameter sanitizer should be final. | 66 | |
Parameter s should be final. | 71 | |
Parameter id should be final. | 71 | |
Parameter s should be final. | 76 | |
Parameter value should be final. | 76 | |
Parameter s1 should be final. | 81 | |
Parameter s2 should be final. | 81 | |
Parameter textIntWritableEdge should be final. | 82 | |
Method 'createVertexReader' is not designed for extension - needs to be abstract, final or empty. | 88 | |
Parameter split should be final. | 90 | |
Parameter context should be final. | 91 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 49 | |
Variable 'textInputFormat' must be private and have accessor methods. | 50 | |
First sentence should end with a period. | 64 | |
First sentence should end with a period. | 66 | |
Parameter lineRecordReader should be final. | 75 | |
'lineRecordReader' hides a field. | 75 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 79 | |
Parameter inputSplit should be final. | 80 | |
Parameter context should be final. | 80 | |
'context' hides a field. | 80 | |
Method 'close' is not designed for extension - needs to be abstract, final or empty. | 86 | |
Method 'getProgress' is not designed for extension - needs to be abstract, final or empty. | 91 | |
Method 'getRecordReader' is not designed for extension - needs to be abstract, final or empty. | 101 | |
Method 'getContext' is not designed for extension - needs to be abstract, final or empty. | 110 | |
Method 'getSplits' is not designed for extension - needs to be abstract, final or empty. | 115 | |
Parameter context should be final. | 116 | |
Parameter numWorkers should be final. | 116 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 46 | |
Variable 'textOutputFormat' must be private and have accessor methods. | 47 | |
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 63 | |
Parameter lineRecordWriter should be final. | 71 | |
'lineRecordWriter' hides a field. | 71 | |
Method 'initialize' is not designed for extension - needs to be abstract, final or empty. | 75 | |
Parameter context should be final. | 76 | |
'context' hides a field. | 76 | |
Method 'close' is not designed for extension - needs to be abstract, final or empty. | 80 | |
Parameter context should be final. | 81 | |
'context' hides a field. | 81 | |
Method 'getRecordWriter' is not designed for extension - needs to be abstract, final or empty. | 91 | |
Method 'getContext' is not designed for extension - needs to be abstract, final or empty. | 100 | |
Method 'checkOutputSpecs' is not designed for extension - needs to be abstract, final or empty. | 105 | |
Parameter context should be final. | 106 | |
Method 'getOutputCommitter' is not designed for extension - needs to be abstract, final or empty. | 111 | |
Parameter context should be final. | 112 |
Violation | Message | Line |
---|---|---|
Class AnnotationUtils should be declared as final. | 35 | |
Parameter annotation should be final. | 50 | |
Parameter packageName should be final. | 50 | |
Parameter packageName should be final. | 68 | |
First sentence should end with a period. | 106 | |
First sentence should end with a period. | 108 | |
First sentence should end with a period. | 110 | |
First sentence should end with a period. | 112 | |
Parameter packageName should be final. | 119 | |
Expected @throws tag for 'IOException'. | 119 | |
First sentence should end with a period. | 146 | |
First sentence should end with a period. | 185 | |
First sentence should end with a period. | 187 | |
Parameter packageName should be final. | 193 | |
Parameter directory should be final. | 210 | |
Parameter files should be final. | 210 | |
First sentence should end with a period. | 239 | |
Parameter className should be final. | 245 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 40 | |
First sentence should end with a period. | 43 | |
First sentence should end with a period. | 46 | |
Parameter arrayListWritable should be final. | 59 | |
Parameter refClass should be final. | 68 | |
'refClass' hides a field. | 68 | |
First sentence should end with a period. | 73 | |
Method 'setClass' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Parameter refClass should be final. | 78 | |
'refClass' hides a field. | 78 | |
'+' should be on a new line. | 81 | |
Method 'readFields' is not designed for extension - needs to be abstract, final or empty. | 93 | |
Parameter in should be final. | 94 | |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 107 | |
Parameter out should be final. | 108 | |
Parameter conf should be final. | 122 | |
'conf' hides a field. | 122 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 24 | |
Class CollectionUtils should be declared as final. | 25 | |
Parameter key should be final. | 41 | |
Parameter values should be final. | 42 | |
Parameter map should be final. | 42 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 23 | |
Class ComparisonUtils should be declared as final. | 24 | |
First sentence should end with a period. | 26 | |
First sentence should end with a period. | 29 | |
Parameter first should be final. | 37 | |
Parameter second should be final. | 37 | |
First sentence should end with a period. | 41 | |
Parameter first should be final. | 49 | |
Parameter second should be final. | 49 | |
Avoid inline conditionals. | 54 | |
':' should be on a new line. | 54 |
Violation | Message | Line |
---|---|---|
Method 'iterator' is not designed for extension - needs to be abstract, final or empty. | 30 | |
Method 'hasNext' is not designed for extension - needs to be abstract, final or empty. | 35 | |
Method 'next' is not designed for extension - needs to be abstract, final or empty. | 40 | |
Method 'remove' is not designed for extension - needs to be abstract, final or empty. | 45 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 29 | |
Method 'getMilliseconds' is not designed for extension - needs to be abstract, final or empty. | 32 | |
Method 'getNanoseconds' is not designed for extension - needs to be abstract, final or empty. | 37 | |
Method 'getSeconds' is not designed for extension - needs to be abstract, final or empty. | 42 | |
Method 'getCurrentDate' is not designed for extension - needs to be abstract, final or empty. | 47 | |
Method 'sleep' is not designed for extension - needs to be abstract, final or empty. | 52 | |
Parameter milliseconds should be final. | 53 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 33 | |
Class FileUtils should be declared as final. | 36 | |
First sentence should end with a period. | 38 | |
Parameter vertexClass should be final. | 50 | |
Expected @throws tag for 'IOException'. | 51 | |
'+' should be on a new line. | 54 | |
Parameter parent should be final. | 71 | |
Parameter name should be final. | 71 | |
Expected @throws tag for 'IOException'. | 72 | |
Parameter parent should be final. | 84 | |
Parameter name should be final. | 84 | |
Expected @throws tag for 'IOException'. | 85 | |
Parameter parent should be final. | 100 | |
Parameter name should be final. | 100 | |
Parameter dir should be final. | 101 | |
Expected @throws tag for 'IOException'. | 101 | |
Parameter file should be final. | 117 | |
Parameter lines should be final. | 117 | |
Expected @throws tag for 'IOException'. | 118 | |
First sentence should end with a period. | 130 | |
Parameter dir should be final. | 135 | |
Parameter f should be final. | 146 | |
Parameter conf should be final. | 162 | |
Parameter path should be final. | 162 | |
Expected @throws tag for 'IOException'. | 163 | |
Parameter conf should be final. | 174 | |
Parameter path should be final. | 174 | |
Expected @throws tag for 'IOException'. | 175 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 31 | |
First sentence should end with a period. | 33 | |
First sentence should end with a period. | 35 | |
Method 'add' is not designed for extension - needs to be abstract, final or empty. | 44 | |
Parameter key should be final. | 44 | |
First sentence should end with a period. | 64 | |
Method 'cardinality' is not designed for extension - needs to be abstract, final or empty. | 69 | |
First sentence should end with a period. | 74 | |
Method 'size' is not designed for extension - needs to be abstract, final or empty. | 79 | |
First sentence should end with a period. | 83 | |
Method 'has' is not designed for extension - needs to be abstract, final or empty. | 89 | |
Parameter key should be final. | 89 | |
Method 'getLastBaseKey' is not designed for extension - needs to be abstract, final or empty. | 104 | |
First sentence should end with a period. | 108 | |
Parameter remainder should be final. | 113 | |
'+' should be on a new line. | 116 | |
'+' should be on a new line. | 117 | |
'+' should be on a new line. | 118 |
Violation | Message | Line |
---|---|---|
Class InternalVertexRunner should be declared as final. | 49 | |
First sentence should end with a period. | 50 | |
Parameter vertexClass should be final. | 71 | |
Parameter vertexInputFormatClass should be final. | 72 | |
Parameter vertexOutputFormatClass should be final. | 72 | |
Parameter params should be final. | 73 | |
Parameter data should be final. | 73 | |
Expected @throws tag for 'Exception'. | 73 | |
Parameter vertexClass should be final. | 92 | |
Parameter vertexCombinerClass should be final. | 93 | |
Parameter vertexInputFormatClass should be final. | 93 | |
Parameter vertexOutputFormatClass should be final. | 94 | |
Parameter params should be final. | 94 | |
Parameter data should be final. | 95 | |
Expected @throws tag for 'Exception'. | 95 | |
'100.0f' is a magic number. | 120 | |
'+' should be on a new line. | 124 | |
First sentence should end with a period. | 189 |
Violation | Message | Line |
---|---|---|
Class MemoryUtils should be declared as final. | 24 | |
First sentence should end with a period. | 28 | |
'+' should be on a new line. | 34 | |
'1024f' is a magic number. | 35 | |
'1024f' is a magic number. | 35 | |
'+' should be on a new line. | 35 | |
'+' should be on a new line. | 36 | |
'1024f' is a magic number. | 37 | |
'1024f' is a magic number. | 37 | |
'+' should be on a new line. | 37 | |
'+' should be on a new line. | 38 | |
'1024f' is a magic number. | 39 | |
'1024f' is a magic number. | 39 |
Violation | Message | Line |
---|---|---|
Class ReflectionUtils should be declared as final. | 38 | |
Parameter type should be final. | 51 | |
Parameter baseClass should be final. | 80 | |
Parameter childClass should be final. | 80 | |
'!' is followed by whitespace. | 84 | |
Parameter target should be final. | 135 | |
Parameter fieldname should be final. | 135 | |
Parameter value should be final. | 135 | |
Expected @throws tag for 'NoSuchFieldException'. | 136 | |
Expected @throws tag for 'IllegalAccessException'. | 136 | |
First sentence should end with a period. | 142 | |
Parameter inClass should be final. | 150 | |
Parameter fieldname should be final. | 150 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 28 | |
Method 'getMilliseconds' is not designed for extension - needs to be abstract, final or empty. | 33 | |
Method 'getNanoseconds' is not designed for extension - needs to be abstract, final or empty. | 38 | |
Method 'getSeconds' is not designed for extension - needs to be abstract, final or empty. | 43 | |
Method 'getCurrentDate' is not designed for extension - needs to be abstract, final or empty. | 48 | |
Method 'sleep' is not designed for extension - needs to be abstract, final or empty. | 53 | |
Parameter milliseconds should be final. | 54 | |
First sentence should end with a period. | 58 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 28 | |
First sentence should end with a period. | 30 | |
First sentence should end with a period. | 32 | |
First sentence should end with a period. | 34 | |
First sentence should end with a period. | 36 | |
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 40 | |
First sentence should end with a period. | 42 | |
First sentence should end with a period. | 44 | |
First sentence should end with a period. | 46 | |
First sentence should end with a period. | 49 | |
First sentence should end with a period. | 58 | |
First sentence should end with a period. | 66 | |
First sentence should end with a period. | 74 | |
First sentence should end with a period. | 81 | |
Expected @throws tag for 'InterruptedException'. | 87 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 27 | |
First sentence should end with a period. | 29 | |
First sentence should end with a period. | 31 | |
First sentence should end with a period. | 34 | |
Parameter msecs should be final. | 40 | |
'msecs' hides a field. | 40 | |
Parameter log should be final. | 40 | |
'log' hides a field. | 40 | |
Method 'info' is not designed for extension - needs to be abstract, final or empty. | 50 | |
Parameter msg should be final. | 50 | |
Method 'isPrintable' is not designed for extension - needs to be abstract, final or empty. | 61 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 24 | |
First sentence should end with a period. | 29 | |
First sentence should end with a period. | 31 | |
Parameter intArray should be final. | 39 | |
'intArray' hides a field. | 39 | |
Method 'hasNext' is not designed for extension - needs to be abstract, final or empty. | 44 | |
Method 'next' is not designed for extension - needs to be abstract, final or empty. | 49 |
Violation | Message | Line |
---|---|---|
Class WritableUtils should be declared as final. | 43 | |
Parameter byteArray should be final. | 56 | |
Parameter writableObject should be final. | 56 | |
Parameter zkExt should be final. | 76 | |
Parameter zkPath should be final. | 77 | |
Parameter watch should be final. | 78 | |
Parameter stat should be final. | 79 | |
Parameter writableObject should be final. | 80 | |
Parameter writableObject should be final. | 99 | |
Parameter zkExt should be final. | 121 | |
Parameter zkPath should be final. | 122 | |
Parameter version should be final. | 123 | |
Parameter writableObject should be final. | 124 | |
Parameter writableList should be final. | 149 | |
Parameter zkExt should be final. | 174 | |
Parameter zkPath should be final. | 175 | |
Parameter version should be final. | 176 | |
Parameter writableList should be final. | 177 | |
Parameter byteArray should be final. | 204 | |
Parameter writableClass should be final. | 205 | |
Parameter conf should be final. | 206 | |
Parameter zkExt should be final. | 237 | |
Parameter zkPath should be final. | 238 | |
Parameter watch should be final. | 239 | |
Parameter stat should be final. | 240 | |
Parameter writableClass should be final. | 241 | |
Parameter conf should be final. | 242 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 36 | |
First sentence should end with a period. | 38 | |
First sentence should end with a period. | 40 | |
Variable 'progressable' must be private and have accessor methods. | 41 | |
First sentence should end with a period. | 42 | |
First sentence should end with a period. | 44 | |
First sentence should end with a period. | 46 | |
First sentence should end with a period. | 48 | |
First sentence should end with a period. | 50 | |
Parameter progressable should be final. | 58 | |
'progressable' hides a field. | 58 | |
Parameter progressable should be final. | 69 | |
'progressable' hides a field. | 69 | |
Parameter msecPeriod should be final. | 69 | |
'msecPeriod' hides a field. | 69 | |
Parameter time should be final. | 69 | |
'time' hides a field. | 69 | |
Method 'reset' is not designed for extension - needs to be abstract, final or empty. | 75 | |
Method 'signal' is not designed for extension - needs to be abstract, final or empty. | 85 | |
Method 'waitMsecs' is not designed for extension - needs to be abstract, final or empty. | 96 | |
Parameter msecs should be final. | 97 | |
'+' should be on a new line. | 115 | |
'+' should be on a new line. | 120 | |
'+' should be on a new line. | 121 | |
Method 'waitForever' is not designed for extension - needs to be abstract, final or empty. | 136 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 42 | |
First sentence should end with a period. | 44 | |
First sentence should end with a period. | 47 | |
Parameter connectString should be final. | 64 | |
Parameter sessionTimeout should be final. | 65 | |
Parameter watcher should be final. | 66 | |
Expected @throws tag for 'IOException'. | 66 | |
Method 'createExt' is not designed for extension - needs to be abstract, final or empty. | 84 | |
Parameter data should be final. | 86 | |
Parameter acl should be final. | 87 | |
Parameter createMode should be final. | 88 | |
Parameter recursive should be final. | 89 | |
Expected @throws tag for 'KeeperException'. | 89 | |
Expected @throws tag for 'InterruptedException'. | 89 | |
'+' should be on a new line. | 113 | |
First sentence should end with a period. | 121 | |
First sentence should end with a period. | 125 | |
First sentence should end with a period. | 127 | |
First sentence should end with a period. | 130 | |
Parameter path should be final. | 136 | |
'path' hides a field. | 136 | |
Parameter stat should be final. | 136 | |
'stat' hides a field. | 136 | |
Method 'getPath' is not designed for extension - needs to be abstract, final or empty. | 146 | |
First sentence should end with a period. | 150 | |
Method 'getStat' is not designed for extension - needs to be abstract, final or empty. | 155 | |
Method 'createOrSetExt' is not designed for extension - needs to be abstract, final or empty. | 173 | |
Parameter data should be final. | 174 | |
Parameter acl should be final. | 175 | |
Parameter createMode should be final. | 176 | |
Parameter recursive should be final. | 177 | |
Parameter version should be final. | 178 | |
Expected @throws tag for 'KeeperException'. | 178 | |
Expected @throws tag for 'InterruptedException'. | 178 | |
First sentence should end with a period. | 192 | |
Method 'createOnceExt' is not designed for extension - needs to be abstract, final or empty. | 204 | |
Parameter data should be final. | 205 | |
Parameter acl should be final. | 206 | |
Parameter createMode should be final. | 207 | |
Parameter recursive should be final. | 208 | |
Expected @throws tag for 'KeeperException'. | 208 | |
Expected @throws tag for 'InterruptedException'. | 208 | |
Method 'deleteExt' is not designed for extension - needs to be abstract, final or empty. | 231 | |
Parameter version should be final. | 231 | |
Parameter recursive should be final. | 231 | |
Expected @throws tag for 'InterruptedException'. | 232 | |
Expected @throws tag for 'KeeperException'. | 232 | |
Method 'getChildrenExt' is not designed for extension - needs to be abstract, final or empty. | 268 | |
Parameter watch should be final. | 270 | |
Parameter sequenceSorted should be final. | 271 | |
Parameter fullPath should be final. | 272 | |
Expected @throws tag for 'KeeperException'. | 272 | |
Expected @throws tag for 'InterruptedException'. | 272 | |
Parameter s1 should be final. | 277 | |
Parameter s2 should be final. | 277 | |
'||' should be on a new line. | 278 | |
'+' should be on a new line. | 281 | |
'+' should be on a new line. | 282 | |
'+' should be on a new line. | 283 | |
'+' should be on a new line. | 284 | |
'-' should be on a new line. | 288 | |
'-' should be on a new line. | 291 |
Violation | Message | Line |
---|---|---|
First sentence should end with a period. | 59 | |
First sentence should end with a period. | 61 | |
First sentence should end with a period. | 63 | |
First sentence should end with a period. | 66 | |
First sentence should end with a period. | 68 | |
First sentence should end with a period. | 70 | |
First sentence should end with a period. | 72 | |
First sentence should end with a period. | 74 | |
First sentence should end with a period. | 76 | |
First sentence should end with a period. | 81 | |
First sentence should end with a period. | 86 | |
First sentence should end with a period. | 88 | |
First sentence should end with a period. | 90 | |
First sentence should end with a period. | 92 | |
First sentence should end with a period. | 94 | |
First sentence should end with a period. | 96 | |
First sentence should end with a period. | 98 | |
First sentence should end with a period. | 100 | |
First sentence should end with a period. | 102 | |
First sentence should end with a period. | 104 | |
First sentence should end with a period. | 106 | |
First sentence should end with a period. | 108 | |
First sentence should end with a period. | 110 | |
First sentence should end with a period. | 112 | |
First sentence should end with a period. | 118 | |
First sentence should end with a period. | 120 | |
First sentence should end with a period. | 122 | |
Parameter context should be final. | 132 | |
'context' hides a field. | 132 | |
Expected @throws tag for 'IOException'. | 133 | |
'+' should be on a new line. | 146 | |
'+' should be on a new line. | 156 | |
'+' should be on a new line. | 159 | |
First sentence should end with a period. | 173 | |
First sentence should end with a period. | 182 | |
Parameter conf should be final. | 190 | |
'+' should be on a new line. | 193 | |
First sentence should end with a period. | 204 | |
First sentence should end with a period. | 206 | |
'is' hides a field. | 214 | |
Method 'setup' is not designed for extension - needs to be abstract, final or empty. | 243 | |
Expected @throws tag for 'IOException'. | 243 | |
Expected @throws tag for 'InterruptedException'. | 243 | |
Method 'createCandidateStamp' is not designed for extension - needs to be abstract, final or empty. | 252 | |
'+' should be on a new line. | 255 | |
'+' should be on a new line. | 258 | |
'+' should be on a new line. | 265 | |
'+' should be on a new line. | 270 | |
'+' should be on a new line. | 271 | |
'+' should be on a new line. | 272 | |
'+' should be on a new line. | 273 | |
'+' should be on a new line. | 278 | |
'+' should be on a new line. | 282 | |
'+' should be on a new line. | 287 | |
'+' should be on a new line. | 299 | |
'+' should be on a new line. | 303 | |
Method 'computationDone' is not designed for extension - needs to be abstract, final or empty. | 312 | |
Expected @throws tag for 'IOException'. | 327 | |
Expected @throws tag for 'InterruptedException'. | 328 | |
'+' should be on a new line. | 341 | |
'+' should be on a new line. | 342 | |
'+' should be on a new line. | 351 | |
'+' should be on a new line. | 352 | |
'+' should be on a new line. | 353 | |
'+' should be on a new line. | 354 | |
'+' should be on a new line. | 355 | |
'+' should be on a new line. | 356 | |
':' should be on a new line. | 370 | |
'+' should be on a new line. | 372 | |
'+' should be on a new line. | 373 | |
'+' should be on a new line. | 382 | |
Expected @throws tag for 'IOException'. | 395 | |
Expected @throws tag for 'IOException'. | 415 | |
Expected @throws tag for 'InterruptedException'. | 416 | |
'+' should be on a new line. | 429 | |
'+' should be on a new line. | 430 | |
'+' should be on a new line. | 431 | |
'+' should be on a new line. | 440 | |
'+' should be on a new line. | 450 | |
'+' should be on a new line. | 451 | |
'+' should be on a new line. | 456 | |
'+' should be on a new line. | 457 | |
'+' should be on a new line. | 458 | |
First sentence should end with a period. | 475 | |
Method 'getZooKeeperServerPortString' is not designed for extension - needs to be abstract, final or empty. | 479 | |
Parameter serverList should be final. | 489 | |
'+' should be on a new line. | 491 | |
'+' should be on a new line. | 492 | |
'+' should be on a new line. | 499 | |
'+' should be on a new line. | 505 | |
'+' should be on a new line. | 510 | |
'+' should be on a new line. | 516 | |
'+' should be on a new line. | 523 | |
'+' should be on a new line. | 527 | |
'+' should be on a new line. | 528 | |
'+' should be on a new line. | 530 | |
'+' should be on a new line. | 531 | |
'+' should be on a new line. | 533 | |
'+' should be on a new line. | 534 | |
'+' should be on a new line. | 536 | |
'+' should be on a new line. | 538 | |
'+' should be on a new line. | 540 | |
'+' should be on a new line. | 545 | |
'+' should be on a new line. | 546 | |
Method 'onlineZooKeeperServers' is not designed for extension - needs to be abstract, final or empty. | 572 | |
Method length is 185 lines (max allowed is 150). | 572 | |
'+' should be on a new line. | 578 | |
'+' should be on a new line. | 583 | |
'+' should be on a new line. | 613 | |
'+' should be on a new line. | 614 | |
'+' should be on a new line. | 628 | |
'+' should be on a new line. | 629 | |
'+' should be on a new line. | 638 | |
'+' should be on a new line. | 650 | |
'+' should be on a new line. | 651 | |
'+' should be on a new line. | 652 | |
'+' should be on a new line. | 653 | |
'+' should be on a new line. | 654 | |
'5000' is a magic number. | 660 | |
'+' should be on a new line. | 662 | |
'+' should be on a new line. | 667 | |
'+' should be on a new line. | 670 | |
'+' should be on a new line. | 673 | |
'+' should be on a new line. | 681 | |
'+' should be on a new line. | 687 | |
'+' should be on a new line. | 691 | |
'+' should be on a new line. | 695 | |
'+' should be on a new line. | 700 | |
'&&' should be on a new line. | 711 | |
'+' should be on a new line. | 719 | |
'+' should be on a new line. | 720 | |
'+' should be on a new line. | 726 | |
'+' should be on a new line. | 727 | |
'+' should be on a new line. | 728 | |
'+' should be on a new line. | 729 | |
'+' should be on a new line. | 730 | |
'+' should be on a new line. | 731 | |
'+' should be on a new line. | 732 | |
'+' should be on a new line. | 733 | |
'+' should be on a new line. | 741 | |
'+' should be on a new line. | 742 | |
'+' should be on a new line. | 751 | |
Parameter totalMapTasks should be final. | 763 | |
'+' should be on a new line. | 779 | |
'+' should be on a new line. | 780 | |
'+' should be on a new line. | 781 | |
'+' should be on a new line. | 782 | |
Method 'offlineZooKeeperServers' is not designed for extension - needs to be abstract, final or empty. | 807 | |
Parameter state should be final. | 807 | |
'+' should be on a new line. | 824 | |
'+' should be on a new line. | 828 | |
'+' should be on a new line. | 833 | |
Method 'runsZooKeeper' is not designed for extension - needs to be abstract, final or empty. | 847 |