You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2022/07/25 19:16:26 UTC

[GitHub] [iceberg] rdblue commented on pull request #5159: Drop table's root folder if the table owns it

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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org