rdblue commented on issue #253: Register existing tables in Iceberg HiveCatalog URL: https://github.com/apache/incubator-iceberg/pull/253#issuecomment-511029823 @aokolnychyi, this looks ready. But before we commit it, I want to suggest an alternative for you to consider. What about instead of adding the `registerTable` method, we added a way to set an existing table's metadata location directly? That would avoid extra public methods in the `Catalog` interface. It would also be a way to roll back the entire table state, not just rolling back the current snapshot. This would primarily be an administrator change, so we could expose it through a method in `TableOperations`, like `setMetadataLocation`. What do you think? This may be difficult with the new table UUID...
---------------------------------------------------------------- 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]
