rdblue commented on PR #5159: URL: https://github.com/apache/iceberg/pull/5159#issuecomment-1194508860
@gaborkaszab, by adding a table property you're defining ownership. The problem is, you're adding an ownership concept that we know is going to change (we will need multiple roots) and doing it with very specific support (as @pvary mentioned, the assumption is that the directory is owned AND that no files are outside of it). As I said, I think the solution is to add a real concept of location ownership rather than adding temporary work-arounds. What about updating the spec to have a list of owned locations and using that to fix this? -- 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]
