Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/23439 )
Change subject: IMPALA-14454: Exclude log4j 2 dependencies ...................................................................... Patch Set 4: Code-Review+1 This looks good to me. I'm trying to think about what testing we need to do. Regular precommit would see obvious issues. Should we run an s3 job or anything else? -- To view, visit http://gerrit.cloudera.org:8080/23439 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic060bdd969a6e5cd01646376b27c7355ce841819 Gerrit-Change-Number: 23439 Gerrit-PatchSet: 4 Gerrit-Owner: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Comment-Date: Tue, 23 Sep 2025 17:43:05 +0000 Gerrit-HasComments: No
