Soumo-git-hub opened a new pull request, #4088: URL: https://github.com/apache/iceberg-python/pull/4088
Closes #4073 # Rationale for this change REST view responses include `metadata-location`, but PyIceberg currently drops this value when constructing a `View`. As a result, callers cannot access the metadata file location returned by the REST catalog. This change preserves the response value on `View.metadata_location`. # What changed - Add an optional `metadata_location` attribute to `View` while preserving the existing constructor argument order. - Propagate `ViewResponse.metadata_location` through `RestCatalog._response_to_view`. - Add regression coverage for `create_view`, `load_view`, and the shared `register_view` response path. ## Are these changes tested? Yes. - `python -m pytest tests/catalog/test_rest.py -m unit -v -p no:cacheprovider -k "test_create_view_200 or test_load_view_200 or test_register_view_200"` — **3 passed** - `python -m pytest tests/catalog/test_rest.py -m unit -v -p no:cacheprovider` — **167 passed** ## Are there any user-facing changes? Yes. `View` now exposes the metadata location returned by the REST catalog through `View.metadata_location`. -- 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]
