[HBASE-22906] Clean up checkstyle complaints in hbase-operator-tools

• Type: Task
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: hbase-operator-tools-1.0.0
• Fix Version/s: hbase-operator-tools-1.0.0
• Component/s: hbase-operator-tools
• Assignee: Sean Busbey
• Reporter: Sean Busbey
• Created: 23/Aug/19 14:46
• Updated: 23/Aug/19 15:30
• Resolved: 23/Aug/19 15:30

Description

easy import fixes:

[INFO] --- maven-checkstyle-plugin:3.0.0:check (default-cli) @ hbase-hbck2 ---
[INFO] There are 2 errors reported by Checkstyle 8.11 with hbase/checkstyle.xml ruleset.
[ERROR] src/main/java/org/apache/hbase/HBCK2.java:[52] (imports) ImportOrder: Wrong order for 'org.apache.logging.log4j.Level' import.
[ERROR] src/test/java/org/apache/hbase/TestSchedulingRecoveries.java:[30] (imports) ImportOrder: Wrong order for 'org.junit.AfterClass' import.
[INFO] ------------------------------------------------------------------------

little confusing because hbase-operator-tools uses the checkstyle rules from version 2.2.0 of the main repo, but the order is supposed to be "normal stuff, then hbase.thirdparty, then hbase.shaded".

Issue Links

• links to: GitHub Pull Request #17