[HBASE-20749] Upgrade our use of checkstyle to 8.6+

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 2.2.0, 2.1.1
• Component/s: build, community
• Assignee: Mike Drob
• Reporter: Sean Busbey
• Created: 18/Jun/18 18:23
• Updated: 31/Aug/18 14:24
• Resolved: 31/Aug/18 14:24

Description

We should upgrade our checkstyle version to 8.6 or later so we can use the "match violation message to this regex" feature for suppression. That will allow us to make sure we don't regress on HTrace v3 vs v4 APIs (came up in HBASE-20332).

We're currently blocked on upgrading to 8.3+ by checkstyle #5279, a regression that flags our use of both the "separate import groups" and "put static imports over here" configs as an error.

Issue Links

• is related to: HBASE-20332 shaded mapreduce module shouldn't include hadoop
• links to: Review Board (master)