[ZOOKEEPER-4058] Update checkstyle-strict.xml by the latest version 8.39 of checkstyle

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.7.0, 3.8.0
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: LiAoNan
• Created: 14/Jan/21 08:35
• Updated: 28/Mar/21 08:54
• Resolved: 17/Jan/21 20:23

Description

1.update checkstyle to latest version 8.39
See https://checkstyle.sourceforge.io/config_javadoc.html

• JavadocMethod: remove properties `allowMissingJavadoc,allowMissingThrowsTags,allowThrowsTagsForSubclasses,allowUndeclaredRTE`

• LineLength: change it's parent to Checker

2.update XML dtd: `checkstyle-strict.xml, checkstyle-simple.xml, checkstyleSuppressions.xml`

3.fix code style:

• `QuorumPeer.java, PemReader.java`. Operators like + and ? appear at newlines rather than at the end of the previous line.
• license of `TestApacheCuratorCompatibility.java`. Checkstyle JavadocParagraph:`<p>` tag should be placed immediately before the first word.

Issue Links

• links to: GitHub Pull Request #1579