szehon-ho commented on PR #5327: URL: https://github.com/apache/iceberg/pull/5327#issuecomment-1203037586
So still to me it makess sense to add a force option to registerTable. Initially we were debating between adding a new Catalog API , or re-using registerTable with force option. I initially preferred the first as its more clear to user, but I guess @rdblue is not a fan of adding the new API. So we could do this, today we throw an error if the table exists, but if force==true, we just call drop table. And clarify in the docs what force option does. I would probably also in the code move some the logic to read the metadataFile up and then add a Precondition that the file exists, before the if (table.exists() block) because now it will drop if force=true. @RussellSpitzer @flyrain does that sound like it will work ? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
