rangareddy commented on code in PR #19490:
URL: https://github.com/apache/hudi/pull/19490#discussion_r3712409176
##########
packaging/hudi-presto-bundle/pom.xml:
##########
@@ -191,11 +191,6 @@
<artifactId>hudi-common</artifactId>
<version>${project.version}</version>
</dependency>
- <dependency>
- <groupId>org.apache.hudi</groupId>
- <artifactId>hudi-hadoop-mr-bundle</artifactId>
- <version>${project.version}</version>
- </dependency>
Review Comment:
Follow-up filed as #19511, covering both bundles with the two-path jar +
reduced-POM recipe from this thread as the required evidence. One thing that
turned up while writing it, worth recording here: unlike the presto bundle,
both of those reach `hudi-hadoop-mr` and `hudi-hadoop-common` through
`hudi-hive-sync` rather than through the fat jar, so removing the dependency
there should not empty their `artifactSet` — but that still needs measuring
rather than assuming, which is the whole lesson of this round.
--
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]
