HonahX commented on issue #270:
URL: https://github.com/apache/iceberg-python/issues/270#issuecomment-1965963701

   Based on offline discussion with @Fokko, I will focus on implementing the 
`MergeAppend` which supports these keys
   - `commit.manifest-merge.enabled`
   - `commit.manifest.min-count-to-merge`
   - `commit.manifest.target-size-bytes`
   
   The MergeAppend will become the default append method since 
`commit.manifest-merge.enabled` is default to `True`.
   The PR for MergeAppend is https://github.com/apache/iceberg-python/pull/363
   
   BTW, it seems `rewrite_manifest` operations only depends on the 
`commit.manifest.target-size-bytes`. Shall we update the description to reflect 
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-unsubscr...@iceberg.apache.org

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


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

Reply via email to