[HBASE-19262] Revisit checkstyle rules

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.4.0, 1.3.2, 2.0.0-beta-1, 2.0.0, 1.2.7
• Component/s: build
• Hadoop Flags: Reviewed
• Release Note: Change the import order rule that now we should put the shaded import at bottom. Ignore the VisibilityModifier warnings for test code.
• Assignee: Duo Zhang
• Reporter: Duo Zhang
• Created: 15/Nov/17 06:56
• Updated: 01/Feb/19 20:19
• Resolved: 16/Nov/17 01:48

Description

For the shaded imports, I prefer we put it at the bottom with a separated block. And also, for UTs which use junit rule, we usually declare the field as public but checkstyle will report a VisibilityModifier problem.

Issue Links

• relates to: HBASE-19327 Update the import layout for formatter