-
Notifications
You must be signed in to change notification settings - Fork 3k
Docs: update about format version table properties #3809
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
site/docs/configuration.md
Outdated
| | --------------------------------------------- | -------- | ------------------------------------------------------------- | | ||
| | compatibility.snapshot-id-inheritance.enabled | false | Enables committing snapshots without explicit snapshot IDs | | ||
|
|
||
| ### Format version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be named "reserved table properties". See the javadoc for more details, but the intention is that there might be other reserved properties in the future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok. updated.
| | Property | Default | Description | | ||
| | -------------- | -------- | ------------------------------------------------------------- | | ||
| | format-version | 1 | Table's format version (can be 1 or 2) | | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about adding a reference to https://iceberg.apache.org/#spec/#format-versioning ? 😸
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Initially I thought of adding it and then I ignored thinking it is understandable. I have updated it now.
|
Thanks, @ajantha-bhat! |
* apache/iceberg#3723 * apache/iceberg#3732 * apache/iceberg#3749 * apache/iceberg#3766 * apache/iceberg#3787 * apache/iceberg#3796 * apache/iceberg#3809 * apache/iceberg#3820 * apache/iceberg#3878 * apache/iceberg#3890 * apache/iceberg#3892 * apache/iceberg#3944 * apache/iceberg#3976 * apache/iceberg#3993 * apache/iceberg#3996 * apache/iceberg#4008 * apache/iceberg#3758 and 3856 * apache/iceberg#3761 * apache/iceberg#2062 * apache/iceberg#3422 * remove restriction related to legacy parquet file list
format version table properties is missing in the configuration docs.