[HBASE-12261] Add checkstyle to HBase build process

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 0.98.8, 0.99.2
• Component/s: build, website
• Hadoop Flags: Reviewed
• Assignee: Elliott Neil Clark
• Reporter: Elliott Neil Clark
• Created: 15/Oct/14 00:58
• Updated: 06/Apr/18 17:55
• Resolved: 17/Oct/14 18:15

Description

We should add checkstyle to hadoop qa for our builds. That would free committers up from checking patches for stylistic issues and leave them free to check the real meat of the patches.

Additionally we should have the check for empty try catch blocks running so that we can't regress on catching exceptions.

Issue Links

• breaks: HBASE-12294 Regression from HBASE-12261: Can't build the docs after the hbase-checkstyle module was added
• breaks: HBASE-12299 Can't assemble tarballs after HBASE-12261