tanishq-chugh commented on code in PR #5404: URL: https://github.com/apache/hive/pull/5404#discussion_r1751408850
########## kudu-handler/pom.xml: ########## @@ -98,6 +98,15 @@ <artifactId>junit-vintage-engine</artifactId> <scope>test</scope> </dependency> + <dependency> + <groupId>org.apache.logging.log4j</groupId> + <artifactId>log4j-1.2-api</artifactId> Review Comment: These logging dependencies are required by upgraded kudu. There has been internal changes in kudu moving from 1.12.0 to 1.17.0 and that requires us to explicitly add these required logging dependencies. -- 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. To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org