[DRILL-6650] Remove Stray Semicolon in Printing Results Listener

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.14.0
• Fix Version/s: 1.14.0, 1.15.0
• Labels: ready-to-commit
• Assignee: Timothy Farkas
• Reporter: Timothy Farkas
• Reviewer: Aman Sinha
• Created: 31/Jul/18 19:17
• Updated: 31/Jul/18 23:20
• Resolved: 31/Jul/18 23:20

Description

Having empty import statements with stray semicolons can cause compilation in eclipse to fail. I investigated adding a checkstyle check to prevent empty import declarations, but apparently there is no way to do this. In fact having multiple semicolons in an import statement is technically not supported by the java spec https://stackoverflow.com/questions/8125558/eclipse-double-semi-colon-on-an-import. There is also a bug filed for the jdk for not throwing an error in this case

https://bugs.openjdk.java.net/browse/JDK-8072390

Since there is no way to automate checking this I am just going to manually remove this error.

Issue Links

• links to: GitHub Pull Request #1410