rdblue commented on a change in pull request #675: Inherit snapshot ids for manifest entries URL: https://github.com/apache/incubator-iceberg/pull/675#discussion_r373283299
########## File path: site/docs/configuration.md ########## @@ -59,6 +59,7 @@ Iceberg tables support table properties to configure table behavior, like the de | commit.manifest.target-size-bytes | 8388608 (8 MB) | Target size when merging manifest files | | commit.manifest.min-count-to-merge | 100 | Minimum number of manifests to accumulate before merging | | commit.manifest-merge.enabled | true | Controls whether to automatically merge manifests on writes | +| commit.snapshot-id-inheritance.enabled | false | Enables committing snapshots without explicit snapshot IDs | Review comment: Can we start a separate section for compatibility flags that explains in more detail? I'd like to highlight that this breaks forward-compatibility and add the minimum supported version. (Doesn't need to happen in this commit) ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
