[GitHub] [incubator-iceberg] aokolnychyi edited a comment on issue #200: Add SnapshotProducer that can rewrite manifests

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

aokolnychyi edited a comment on issue #200: Add SnapshotProducer that can rewrite manifests
URL: https://github.com/apache/incubator-iceberg/pull/200#issuecomment-498380721
 
 
   Am I correct that this API reorders manifests and their content? `MergingSnapshotUpdate` keeps the ordering to efficiently age off data files. Do we expect users to apply this API to recent manifests on a regular basis if their writes frequently modify a lot of partitions?
   
   Also, what will happen if our clustering key assigns a lot of files to one manifest?
   Will `commit.manifest.target-size-bytes` be respected?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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