tomtongue opened a new pull request, #14212: URL: https://github.com/apache/iceberg/pull/14212
The context of this PR is https://github.com/apache/iceberg/pull/12017#issuecomment-3238253488. ## Changes This PR tries to deprecate the `write.metadata.path` property which was added in [PR#12017](https://github.com/apache/iceberg/pull/12017), and use `location` instead to for the view location configuration. ## Background That PR#12017 added `write.metadata.path` to customize the View location. However, the [Iceberg View Spec](https://iceberg.apache.org/view-spec/#view-metadata) defines `location` as its view location. Using `write.metadata.path` is confusing because it specifies two view locations in its view properties, along with `location`. ## Behavior difference between `location` and `write.metadata.path` Both properties can configure the view location, however, * if `write.metadata.path` is specified, the view metadata is put in the path defined `write.metadata.path` * (e.g.) When `'write.metadata.path'='custom-location/path/to` is specified, the view metadata is put in `custom-location/path/to` * if `location` is specified, the view metadata is put in the path defined `location` with `/metadata` path * (e.g.) When `'location='custom-location/path/to` is specified, the view metadata is put in `custom-location/path/to/metadata` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
