You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2019/07/01 18:30:30 UTC

[GitHub] [incubator-iceberg] rdsr commented on issue #236: iceberg: don't bundle hdfs with iceberg runtime

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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org