manishmalhotrawork commented on issue #524: respect commit.manifest.min.count URL: https://github.com/apache/incubator-iceberg/pull/524#issuecomment-541908787 @rdblue Sorry, missed your question ! this PR is trying to resolve #490 Where we add new manifests by calling `appendManifest()` , then the `commit.manifest.min-count-to-merge` property is ignored. This happens because of `MergingSnapshotProducer` checks min count property along with if the `cachedNewManifest` is present. And `cachedNewManifest` is created when new files are added and not manifest files.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
