Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/11987 )
Change subject: IMPALA-7823: Clean up unused Java imports ...................................................................... Patch Set 1: Thanks! Have you looked into any maven plugins or other test-breaking things (like we do with gerrit-code-review-checks for long lines and whitespace and what-not) that find these automatically? https://github.com/diffplug/spotless/tree/master/plugin-maven with only <removeUnusedImports/> seems like something we could try. I've never used it; just found it right now. -- To view, visit http://gerrit.cloudera.org:8080/11987 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic1820a7f9e55dc168510987520949026169f445e Gerrit-Change-Number: 11987 Gerrit-PatchSet: 1 Gerrit-Owner: Paul Rogers <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Paul Rogers <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Comment-Date: Mon, 26 Nov 2018 22:46:47 +0000 Gerrit-HasComments: No
