Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/23432 )
Change subject: IMPALA-14446: Clean up pom.xml ...................................................................... Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/23432/3/fe/pom.xml File fe/pom.xml: http://gerrit.cloudera.org:8080/#/c/23432/3/fe/pom.xml@a198 PS3, Line 198: : : : > Are notes like this important to preserve, or they does not matter anymore? The question is does Ranger include anything besides orc-core, since we include it explicitly. I didn't see evidence they did. It wouldn't hurt to add it back I suppose. http://gerrit.cloudera.org:8080/#/c/23432/3/fe/pom.xml@794 PS3, Line 794: <exclude>com.amazonaws:aws-java-sdk-bundle</exclude> > Why is this added? I'll note it in the commit message. This should be excluded since we use our own minimal s3a jar instead. Just making sure it doesn't accidentally creep in. -- To view, visit http://gerrit.cloudera.org:8080/23432 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If96649840e23036b4a73ee23e8d12516497994f0 Gerrit-Change-Number: 23432 Gerrit-PatchSet: 3 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Wed, 17 Sep 2025 23:03:22 +0000 Gerrit-HasComments: Yes
