aplex opened a new pull request #3320: URL: https://github.com/apache/gobblin/pull/3320
Previously, checkstyle was configured to check only a small subset of things, and imports were not one of them. As a result, reviewers spend their time commenting on code style, when it can be fully verified automatically by CI job. We enable unused/redundant import checks, and fix existing issues, so that the build could complete successfully. The only relevant change is in checkstyle.xml, all other files where changed by IntelliJ "optimize imports" function. For most files it just removed unused imports, but some of them had imports incorrectly ordered, and IntelliJ fixed that as well. https://issues.apache.org/jira/browse/GOBBLIN-1480 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
