viirya commented on a change in pull request #2459:
URL: https://github.com/apache/hive/pull/2459#discussion_r695185072
##########
File path: llap-server/pom.xml
##########
@@ -38,6 +38,7 @@
<groupId>org.apache.hive</groupId>
<artifactId>hive-exec</artifactId>
<version>${project.version}</version>
+ <classifier>core</classifier>
Review comment:
As more dependencies are relocated here, some modules if they depends on
non-core artifact, will cause class not found error...
The motivation is because we want to use shaded version of hive-exec (i.e.,
w/o classifier) in Spark to make sure it doesn't conflict guava version there.
But there are more dependencies conflict with Spark. We need to relocate these
dependencies too..
--
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]
