asolimando commented on a change in pull request #3095:
URL: https://github.com/apache/hive/pull/3095#discussion_r828101962
##########
File path: ql/pom.xml
##########
@@ -1072,6 +1072,7 @@
<artifactSet>
<includes>
<!-- order is meant to be the same as the ant build -->
+ <include>org.antlr:antlr-runtime</include>
Review comment:
It's the Tez worker throwing the classnotfound exception, I don't think
that Hive plays any role on that. From what you said earlier, it's also clear
that nothing has changed on the Hive side regarding the shipping of antlr
classes.
I have checked the jars for tez 0.9.1 and 0.10.0 (in all the locations),
none of them has the antlr classes either, so I am a bit puzzled, they were
either not needed before, or there is something trickier like class loader
issues. I am not sure where to go from here, but I understand your concern.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
