rexminnis opened a new pull request, #16007:
URL: https://github.com/apache/iceberg/pull/16007
## What changes are included in this PR?
Adds `CATALOG_NAME` environment variable support to the REST catalog test
fixture, allowing users to override the backend catalog name without the
unintuitive `CATALOG_CATALOG_NAME=...` form required by the existing
`CATALOG_*` prefix translation.
- `RESTCatalogServer` resolves `CATALOG_NAME` explicitly before the standard
property lookup, using `putIfAbsent` so any caller-provided `catalog.name`
property still wins.
- `docker/iceberg-rest-fixture/README.md` gains a Configuration section
documenting `CATALOG_NAME` and the general `CATALOG_*` env-var scheme.
- New `RESTCatalogServerTest` exercises the helper with 4 cases: env set,
property-wins-over-env, env missing, env empty.
Closes #14972.
## Are these changes tested?
Yes — new `RESTCatalogServerTest` runs as part of `:iceberg-open-api:test`.
## Are there any user-facing changes?
No behavior change when `CATALOG_NAME` is unset — default remains
`rest_backend`. Users can now set `-e CATALOG_NAME=mycatalog` on the
`apache/iceberg-rest-fixture` container to change the catalog name served
by the fixture.
--
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]
