Build failed in Jenkins: PreCommit-ZOOKEEPER-github-pr-build-maven #1665

[Apache Jenkins Server <je...@builds.apache.org>]

See <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/1665/display/redirect?page=changes>

Changes:

[andor] ZOOKEEPER-3539: Fix CI for branch-3.5:


------------------------------------------
[...truncated 4.37 MB...]
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[90,68] (whitespace) WhitespaceAround: '?' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[90,68] (whitespace) WhitespaceAround: '?' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[90,71] (whitespace) WhitespaceAround: ':' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[90,71] (whitespace) WhitespaceAround: ':' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[91] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[92] (sizes) LineLength: Line is longer than 80 characters (found 102).
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[92,52] (whitespace) WhitespaceAround: '?' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[92,56] (whitespace) WhitespaceAround: ':' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[92,74] (whitespace) WhitespaceAround: '<' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[92,74] (whitespace) WhitespaceAround: '<' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[92,96] (whitespace) WhitespaceAround: '?' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[92,96] (whitespace) WhitespaceAround: '?' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[92,99] (whitespace) WhitespaceAround: ':' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[92,99] (whitespace) WhitespaceAround: ':' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[93] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[96,3] (design) DesignForExtension: Class 'LearnerInfo' looks like designed for extension (can be subclassed), but the method 'equals' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'LearnerInfo' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[96,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[96,25] (misc) FinalParameters: Parameter peer_ should be final.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[96,32] (naming) ParameterName: Name 'peer_' must match pattern '^[a-z][a-zA-Z0-9]*$'.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[105,20] (whitespace) WhitespaceAround: '==' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[105,20] (whitespace) WhitespaceAround: '==' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[106] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[107,27] (whitespace) WhitespaceAround: '==' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[107,27] (whitespace) WhitespaceAround: '==' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[108] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[111,3] (design) DesignForExtension: Class 'LearnerInfo' looks like designed for extension (can be subclassed), but the method 'hashCode' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'LearnerInfo' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[111,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[112,18] (coding) MagicNumber: '17' is a magic number.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[114,26] (whitespace) WhitespaceAround: '^' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[114,26] (whitespace) WhitespaceAround: '^' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[114,36] (whitespace) WhitespaceAround: '>>>' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[114,36] (whitespace) WhitespaceAround: '>>>' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[114,39] (coding) MagicNumber: '32' is a magic number.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[115,14] (coding) MagicNumber: '37' is a magic number.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[115,16] (whitespace) WhitespaceAround: '*' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[115,16] (whitespace) WhitespaceAround: '*' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[116,15] (whitespace) WhitespaceAfter: 'typecast' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[117,14] (coding) MagicNumber: '37' is a magic number.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[117,16] (whitespace) WhitespaceAround: '*' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[117,16] (whitespace) WhitespaceAround: '*' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/quorum/LearnerInfo.java:[120,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[1] (javadoc) JavadocPackage: Missing package-info.java file.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[22] (imports) AvoidStarImport: Using the '.*' form of import should be avoided - org.apache.jute.*.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[24] (javadoc) JavadocType: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[26,3] (javadoc) JavadocVariable: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[27,3] (javadoc) JavadocVariable: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[28,3] (javadoc) JavadocVariable: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[29,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[31,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[32,9] (misc) FinalParameters: Parameter magic should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[32,13] (coding) HiddenField: 'magic' hides a field.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[33,9] (misc) FinalParameters: Parameter version should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[33,13] (coding) HiddenField: 'version' hides a field.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[34,9] (misc) FinalParameters: Parameter dbid should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[34,14] (coding) HiddenField: 'dbid' hides a field.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[35,15] (whitespace) WhitespaceAround: '=' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[35,15] (whitespace) WhitespaceAround: '=' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[36,17] (whitespace) WhitespaceAround: '=' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[36,17] (whitespace) WhitespaceAround: '=' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[37,14] (whitespace) WhitespaceAround: '=' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[37,14] (whitespace) WhitespaceAround: '=' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[39,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'getMagic' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'getMagic' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[39,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[42,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'setMagic' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'setMagic' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[42,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[42,24] (misc) FinalParameters: Parameter m_ should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[42,28] (naming) ParameterName: Name 'm_' must match pattern '^[a-z][a-zA-Z0-9]*$'.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[43,10] (whitespace) WhitespaceAround: '=' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[43,10] (whitespace) WhitespaceAround: '=' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[45,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'getVersion' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'getVersion' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[45,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[48,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'setVersion' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'setVersion' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[48,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[48,26] (misc) FinalParameters: Parameter m_ should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[48,30] (naming) ParameterName: Name 'm_' must match pattern '^[a-z][a-zA-Z0-9]*$'.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[49,12] (whitespace) WhitespaceAround: '=' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[49,12] (whitespace) WhitespaceAround: '=' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[51,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'getDbid' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'getDbid' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[51,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[54,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'setDbid' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'setDbid' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[54,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[54,23] (misc) FinalParameters: Parameter m_ should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[54,28] (naming) ParameterName: Name 'm_' must match pattern '^[a-z][a-zA-Z0-9]*$'.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[55,9] (whitespace) WhitespaceAround: '=' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[55,9] (whitespace) WhitespaceAround: '=' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[57] (sizes) LineLength: Line is longer than 80 characters (found 82).
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[57,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'serialize' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'serialize' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[57,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[57,25] (misc) FinalParameters: Parameter a_ should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[57,39] (naming) ParameterName: Name 'a_' must match pattern '^[a-z][a-zA-Z0-9]*$'.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[57,43] (misc) FinalParameters: Parameter tag should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[58,24] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[59,22] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[60,24] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[61,22] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[62,22] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[64] (sizes) LineLength: Line is longer than 80 characters (found 83).
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[64,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'deserialize' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'deserialize' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[64,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[64,27] (misc) FinalParameters: Parameter a_ should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[64,40] (naming) ParameterName: Name 'a_' must match pattern '^[a-z][a-zA-Z0-9]*$'.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[64,44] (misc) FinalParameters: Parameter tag should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[66,10] (whitespace) WhitespaceAround: '=' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[66,10] (whitespace) WhitespaceAround: '=' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[67,12] (whitespace) WhitespaceAround: '=' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[67,12] (whitespace) WhitespaceAround: '=' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[68,9] (whitespace) WhitespaceAround: '=' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[68,9] (whitespace) WhitespaceAround: '=' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[71,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'toString' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[71,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[75] (regexp) RegexpSingleline: Line has trailing spaces.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[75,24] (naming) LocalVariableName: Name 'a_' must match pattern '^[a-z][a-zA-Z0-9]*$'.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[77,26] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[78,22] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[79,24] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[80,22] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[81,24] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[88,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'write' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[88,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[88,21] (misc) FinalParameters: Parameter out should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[92,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'readFields' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'readFields' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[92,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[92,26] (misc) FinalParameters: Parameter in should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[96,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'compareTo' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'compareTo' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[96,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[96,24] (whitespace) MethodParamPad: '(' is preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[96,25] (misc) FinalParameters: Parameter peer_ should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[96,32] (naming) ParameterName: Name 'peer_' must match pattern '^[a-z][a-zA-Z0-9]*$'.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[102,32] (whitespace) WhitespaceAround: '?' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[102,36] (whitespace) WhitespaceAround: ':' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[102,44] (whitespace) WhitespaceAround: '<' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[102,44] (whitespace) WhitespaceAround: '<' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[102,56] (whitespace) WhitespaceAround: '?' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[102,56] (whitespace) WhitespaceAround: '?' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[102,59] (whitespace) WhitespaceAround: ':' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[102,59] (whitespace) WhitespaceAround: ':' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[103] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[104,36] (whitespace) WhitespaceAround: '?' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[104,40] (whitespace) WhitespaceAround: ':' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[104,50] (whitespace) WhitespaceAround: '<' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[104,50] (whitespace) WhitespaceAround: '<' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[104,64] (whitespace) WhitespaceAround: '?' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[104,64] (whitespace) WhitespaceAround: '?' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[104,67] (whitespace) WhitespaceAround: ':' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[104,67] (whitespace) WhitespaceAround: ':' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[105] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[106,30] (whitespace) WhitespaceAround: '?' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[106,34] (whitespace) WhitespaceAround: ':' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[106,41] (whitespace) WhitespaceAround: '<' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[106,41] (whitespace) WhitespaceAround: '<' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[106,52] (whitespace) WhitespaceAround: '?' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[106,52] (whitespace) WhitespaceAround: '?' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[106,55] (whitespace) WhitespaceAround: ':' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[106,55] (whitespace) WhitespaceAround: ':' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[107] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[110,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'equals' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[110,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[110,25] (misc) FinalParameters: Parameter peer_ should be final.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[110,32] (naming) ParameterName: Name 'peer_' must match pattern '^[a-z][a-zA-Z0-9]*$'.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[119,17] (whitespace) WhitespaceAround: '==' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[119,17] (whitespace) WhitespaceAround: '==' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[120] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[121,19] (whitespace) WhitespaceAround: '==' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[121,19] (whitespace) WhitespaceAround: '==' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[122] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[123,16] (whitespace) WhitespaceAround: '==' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[123,16] (whitespace) WhitespaceAround: '==' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[124] (blocks) NeedBraces: 'if' construct must use '{}'s.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[127,3] (design) DesignForExtension: Class 'FileHeader' looks like designed for extension (can be subclassed), but the method 'hashCode' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileHeader' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[127,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[128,18] (coding) MagicNumber: '17' is a magic number.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[130,15] (whitespace) WhitespaceAfter: 'typecast' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[131,14] (coding) MagicNumber: '37' is a magic number.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[131,16] (whitespace) WhitespaceAround: '*' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[131,16] (whitespace) WhitespaceAround: '*' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[132,15] (whitespace) WhitespaceAfter: 'typecast' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[133,14] (coding) MagicNumber: '37' is a magic number.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[133,16] (whitespace) WhitespaceAround: '*' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[133,16] (whitespace) WhitespaceAround: '*' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[134,22] (whitespace) WhitespaceAround: '^' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[134,22] (whitespace) WhitespaceAround: '^' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[134,28] (whitespace) WhitespaceAround: '>>>' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[134,28] (whitespace) WhitespaceAround: '>>>' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[134,31] (coding) MagicNumber: '32' is a magic number.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[135,14] (coding) MagicNumber: '37' is a magic number.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[135,16] (whitespace) WhitespaceAround: '*' is not followed by whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[135,16] (whitespace) WhitespaceAround: '*' is not preceded with whitespace.
[ERROR] target/classes/org/apache/zookeeper/server/persistence/FileHeader.java:[138,3] (javadoc) JavadocMethod: Missing a Javadoc comment.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for Apache ZooKeeper 3.4.14-SNAPSHOT:
[INFO] 
[INFO] Apache ZooKeeper ................................... SUCCESS [ 23.632 s]
[INFO] Apache ZooKeeper - Documentation ................... SUCCESS [  9.939 s]
[INFO] Apache ZooKeeper - Jute ............................ FAILURE [01:18 min]
[INFO] Apache ZooKeeper - Server .......................... SKIPPED
[INFO] Apache ZooKeeper - Client .......................... SKIPPED
[INFO] Apache ZooKeeper - Client - C ...................... SKIPPED
[INFO] Apache ZooKeeper - Recipes ......................... SKIPPED
[INFO] Apache ZooKeeper - Recipes - Election .............. SKIPPED
[INFO] Apache ZooKeeper - Recipes - Lock .................. SKIPPED
[INFO] Apache ZooKeeper - Recipes - Queue ................. SKIPPED
[INFO] Apache ZooKeeper - Contrib ......................... SKIPPED
[INFO] Apache ZooKeeper - Contrib - Loggraph .............. SKIPPED
[INFO] Apache ZooKeeper - Contrib - Rest .................. SKIPPED
[INFO] Apache ZooKeeper - Contrib - ZooInspector .......... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  02:26 min
[INFO] Finished at: 2019-11-28T21:43:14Z
[INFO] ------------------------------------------------------------------------
Waiting for Jenkins to finish collecting data
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.0:check (default-cli) on project zookeeper-jute: You have 11800 Checkstyle violations. -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR] 
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn <goals> -rf :zookeeper-jute
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/pom.xml> to org.apache.zookeeper/zookeeper/3.4.14-SNAPSHOT/zookeeper-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/target/zookeeper-3.4.14-SNAPSHOT-source-package.tar.gz> to org.apache.zookeeper/zookeeper/3.4.14-SNAPSHOT/zookeeper-3.4.14-SNAPSHOT-source-package.tar.gz
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/target/zookeeper-3.4.14-SNAPSHOT-source-package.zip> to org.apache.zookeeper/zookeeper/3.4.14-SNAPSHOT/zookeeper-3.4.14-SNAPSHOT-source-package.zip
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-contrib/zookeeper-contrib-loggraph/pom.xml> to org.apache.zookeeper/zookeeper-contrib-loggraph/3.4.14-SNAPSHOT/zookeeper-contrib-loggraph-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-recipes/zookeeper-recipes-election/pom.xml> to org.apache.zookeeper/zookeeper-recipes-election/3.4.14-SNAPSHOT/zookeeper-recipes-election-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-recipes/zookeeper-recipes-lock/pom.xml> to org.apache.zookeeper/zookeeper-recipes-lock/3.4.14-SNAPSHOT/zookeeper-recipes-lock-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-contrib/pom.xml> to org.apache.zookeeper/zookeeper-contrib/3.4.14-SNAPSHOT/zookeeper-contrib-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-client/pom.xml> to org.apache.zookeeper/zookeeper-client/3.4.14-SNAPSHOT/zookeeper-client-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-jute/pom.xml> to org.apache.zookeeper/zookeeper-jute/3.4.14-SNAPSHOT/zookeeper-jute-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-jute/target/zookeeper-jute-3.4.14-SNAPSHOT.jar> to org.apache.zookeeper/zookeeper-jute/3.4.14-SNAPSHOT/zookeeper-jute-3.4.14-SNAPSHOT.jar
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-jute/target/zookeeper-jute-3.4.14-SNAPSHOT-tests.jar> to org.apache.zookeeper/zookeeper-jute/3.4.14-SNAPSHOT/zookeeper-jute-3.4.14-SNAPSHOT-tests.jar
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-jute/target/zookeeper-jute-3.4.14-SNAPSHOT-sources.jar> to org.apache.zookeeper/zookeeper-jute/3.4.14-SNAPSHOT/zookeeper-jute-3.4.14-SNAPSHOT-sources.jar
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-jute/target/zookeeper-jute-3.4.14-SNAPSHOT-javadoc.jar> to org.apache.zookeeper/zookeeper-jute/3.4.14-SNAPSHOT/zookeeper-jute-3.4.14-SNAPSHOT-javadoc.jar
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-recipes/zookeeper-recipes-queue/pom.xml> to org.apache.zookeeper/zookeeper-recipes-queue/3.4.14-SNAPSHOT/zookeeper-recipes-queue-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-docs/pom.xml> to org.apache.zookeeper/zookeeper-docs/3.4.14-SNAPSHOT/zookeeper-docs-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-docs/target/zookeeper-docs-3.4.14-SNAPSHOT.jar> to org.apache.zookeeper/zookeeper-docs/3.4.14-SNAPSHOT/zookeeper-docs-3.4.14-SNAPSHOT.jar
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-docs/target/zookeeper-docs-3.4.14-SNAPSHOT-tests.jar> to org.apache.zookeeper/zookeeper-docs/3.4.14-SNAPSHOT/zookeeper-docs-3.4.14-SNAPSHOT-tests.jar
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-docs/target/zookeeper-docs-3.4.14-SNAPSHOT-sources.jar> to org.apache.zookeeper/zookeeper-docs/3.4.14-SNAPSHOT/zookeeper-docs-3.4.14-SNAPSHOT-sources.jar
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-recipes/pom.xml> to org.apache.zookeeper/zookeeper-recipes/3.4.14-SNAPSHOT/zookeeper-recipes-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-client/zookeeper-client-c/pom.xml> to org.apache.zookeeper/zookeeper-client-c/3.4.14-SNAPSHOT/zookeeper-client-c-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-contrib/zookeeper-contrib-rest/pom.xml> to org.apache.zookeeper/zookeeper-contrib-rest/3.4.14-SNAPSHOT/zookeeper-contrib-rest-3.4.14-SNAPSHOT.pom
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-server/pom.xml> to org.apache.zookeeper/zookeeper-server/3.4.14-SNAPSHOT/zookeeper-server-3.4.14-SNAPSHOT.pom
[Fast Archiver] No prior successful build to compare, so performing full copy of artifacts
[JENKINS] Archiving <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/ws/zookeeper-contrib/zookeeper-contrib-zooinspector/pom.xml> to org.apache.zookeeper/zookeeper-contrib-zooinspector/3.4.14-SNAPSHOT/zookeeper-contrib-zooinspector-3.4.14-SNAPSHOT.pom
channel stopped
[SpotBugs] Skipping execution of recorder since overall result is 'FAILURE'
Putting comment on the pull request

Jenkins build is back to normal : PreCommit-ZOOKEEPER-github-pr-build-maven #1666

[Apache Jenkins Server <je...@builds.apache.org>]

See <https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build-maven/1666/display/redirect>