[ARROW-1688] [Java] Fail build on checkstyle warnings

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.12.0
• Component/s: Java
• External issue URL: https://github.com/apache/arrow/issues/17697
• Assignee: Bryan Cutler
• Reporter: Wes McKinney
• Created: 19/Oct/17 00:08
• Updated: 11/Jan/23 07:16
• Resolved: 05/Dec/18 18:27

Description

see discussion in ARROW-1474

My plan is to separate the stylecheck fixes into manageable chunks so they are easier to review. I'll do this by enabling the build to fail on style, then suppressing all checks but the ones to be fixed.

Sub-Tasks

• [Java] Style Checks - Fix import ordering: Resolved
• [Java] checkstyle - fix line length and indentation: Resolved
• [Java] checkstyle - fix whitespace: Resolved
• [Java] Document checkstyle deviations from Google style guide: Resolved
• [Java] checkstyle - fix javadoc style: Resolved
• [Java] checkstyle - fix naming: Resolved
• [Java] checkstyle - fix license: Resolved
• [Java] checkstyle - fix remaining coding checks: Resolved