Uploaded image for project: 'Apache Drill'
  1. Apache Drill
  2. DRILL-6650

Remove Stray Semicolon in Printing Results Listener

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.14.0
    • Fix Version/s: 1.14.0, 1.15.0
    • Component/s: None
    • Labels:

      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.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                timothyfarkas Timothy Farkas
                Reporter:
                timothyfarkas Timothy Farkas
                Reviewer:
                Aman Sinha
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: