Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/19146 )
Change subject: IMPALA-11667: Clean up Java dependency exclusions ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/19146/5/java/pom.xml File java/pom.xml: http://gerrit.cloudera.org:8080/#/c/19146/5/java/pom.xml@255 PS5, Line 255: <dependencies> : <!-- Provided by reload4j instead. --> : <dependency> : <groupId>log4j</groupId> : <artifactId>log4j</artifactId> : <version>1.2.17</version> : <scope>provided</scope> : </dependency> : : <!-- Provided by our own s3a-aws-sdk shaded jar. --> : <dependency> : <groupId>com.amazonaws</groupId> : <artifactId>aws-java-sdk-bundle</artifactId> : <version>1.12.132</version> : <scope>provided</scope> : </dependency> : </dependencies> > I think this is a problem for the shaded s3 jar. The minimal shaded s3 jar It does seem error-prone, and with log4j we can't have a check that it's not present. I'll try removing these and using exclusions, hopefully the other changes have made the need for them less pervasive. -- To view, visit http://gerrit.cloudera.org:8080/19146 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I424a175135855dcbd38ae432ea111cca5f562633 Gerrit-Change-Number: 19146 Gerrit-PatchSet: 5 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Tue, 18 Oct 2022 00:06:51 +0000 Gerrit-HasComments: Yes
