rdsr commented on issue #236: iceberg: don't bundle hdfs with iceberg runtime URL: https://github.com/apache/incubator-iceberg/pull/236#issuecomment-507375219 @rdblue there are some changes over here which are not specific to #217, and there are a couple more unrelated changes I'd like to point out[1]. Let me know if you'd like it to club it as part of this PR or make those changes as part of a separate PR and keep this PR specific to dependency exclusion. My fine both ways, if we club it as part of this PR we can update the summary in #217 and this PR [1]. Some other changes I'd like to add - In Presto-runtime we are excluding guava. I think we should include and shade it.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
