laserninja opened a new pull request, #10652:
URL: https://github.com/apache/gravitino/pull/10652
### What changes were proposed in this pull request?
Modified the Iceberg REST API config endpoint to use the `prefix` path
parameter
for catalog resolution. Previously the prefix was ignored and only the
`warehouse`
query parameter was used.
### Why are the changes needed?
The `prefix` in the Iceberg REST API path represents the catalog name. The
config
endpoint should return catalog-specific properties when a prefix is
specified,
instead of always returning the default catalog config.
Fix: #10640
### Does this PR introduce _any_ user-facing change?
Yes. When a prefix is specified in the config endpoint URL (e.g.,
`/v1/{catalog}/config`),
the response now returns the catalog properties for that specific catalog
instead of
the default catalog.
### How was this patch tested?
- Updated existing unit tests to account for prefix-based catalog resolution
- Added `testConfigWithPrefixReturnsCatalogConfig` to verify full catalog
config is returned
- Added `testConfigWithPrefixAndWarehouse` to verify prefix takes precedence
- All tests pass: `./gradlew :iceberg:iceberg-rest-server:test --tests
TestIcebergConfig`
--
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]
