umehrot2 commented on pull request #1944: URL: https://github.com/apache/hudi/pull/1944#issuecomment-673102081
> > 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? > > @vinothchandar @umehrot2 Starting from version 0.233, Presto depends on hudi-hadoop-mr during compile time. Prior to that we needed to drop bundle jars into the plugin directory. > @umehrot2 If this fix is for 0.232 would this go away when picking up 0.233 ? Yeah this is based on my testing for presto 0.232. I did not know the `presto-bundle` would be deprecated going forward. But I am wondering if this was the right decision to depend on `hudi-hadoop-mr` in `presto`. How would we avoid version conflicts like the ones that came up with hbase here ? ---------------------------------------------------------------- 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]
