snazy commented on PR #3719: URL: https://github.com/apache/polaris/pull/3719#issuecomment-3889756275
Honestly, I think this 'overwrite` flag can be dangerous, because it allows users to change an existing table in a breaking way. The ability to replace an existing table should be guarded by a separate privilege. We should also think about what happens to the existing, old (meta)data files and how changes to the base-location and write-(meta)data properties are handled. I do not mind adding this to the feature branch, but it would be really good to have a separate discussion about the behavior before this change goes into the `main` branch. The safest approach IMO would be to initially just error out when `overwrite == true`. -- 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]
