vrozov commented on issue #271: Remove iceberg-presto-runtime that is no longer used. URL: https://github.com/apache/incubator-iceberg/pull/271#issuecomment-514457412 @electrum The PR has 46 files added/modified, and the only relevant to the discussion in this thread class is `io.prestosql.iceberg.HiveTableOperations`, so it was quite easy to miss that file. Anyway, would not it be better to change iceberg `org.apache.iceberg.hive.HiveTableOperations` to allow Presto/Drill/Spark to provide it's own Hive metastore client, instead of re-implementing that class in Presto, especially that `BaseMetastoreTableOperations` that `HiveTableOperations` extends is not part of the iceberg api library?
---------------------------------------------------------------- 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]
