bhasudha commented on pull request #1944: URL: https://github.com/apache/hudi/pull/1944#issuecomment-673103276
> > I am kind of torn. On one hand, this completes the bootstrap story and lets us stabilize things. Given the later versions of presto anyway take hudi as a compile depedency, I think it may be ok? > > cc @bhasudha does prestoDB depend on the bundle or the hoodie-hadoop-mr jars? > > Yes this is a good question. Should we really depend on `hud-hadoop-mr` in presto or the `presto-bundle` ? The bundle gives us the advantages of shading the dependencies to avoid such conflicts. Without bundle we would have to do the shading in `hudi-hadoop-mr` which does not seem right. I agree that doing shading in the `hudi-hadoop-mr` is not right. As far depending on bundle itself, we would need to test the shading in both s3 and HDFS environments to include all right shading if we went that route. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
