[GitHub] [iceberg] danielcweeks edited a comment on pull request #4402: AWS: Add table level S3 tags

[GitBox <gi...@apache.org>]

danielcweeks edited a comment on pull request #4402:
URL: https://github.com/apache/iceberg/pull/4402#issuecomment-1079982601


   I think it would be good to discuss what the use case is for this. I can understand trying to annotate some additional information but I'm not sure that this tracks very well with the evolution of tables. If you rename a table now all of the objects are out of sync.  This also isn't particularly helpful for recovering table data because you actually don't know based on the file what manifests they belong to and that of course can change as well.  
   
   Because iceberg metadata tracks from the metadata file all the way down to the physical file, I'm not sure there's any reason to do this that I can think of.


-- 
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