rexminnis commented on PR #16007: URL: https://github.com/apache/iceberg/pull/16007#issuecomment-4411934436
Thanks @kevinjqliu — agreed on closing #14972; the existing `CATALOG_CATALOG_NAME=…` form works and a parallel code path isn't worth the complexity. I'll drop the Java change and the new test. The docs aren't introducing `CATALOG_WAREHOUSE` as a new variable; it already works via the standard `CATALOG_*` prefix translation. The README spells out the mapping so first-time users don't have to read source. Per the project's [AI-assisted contribution guidelines](https://iceberg.apache.org/contribute/#guidelines-for-ai-assisted-contributions): yes, AI-assisted (draft + test scaffolding); the contribution came from a real interop pain point with the fixture, I understand the `RCKUtils.environmentCatalogConfig` translation path end-to-end, and I reviewed/ran the changes locally. Force-push incoming with the docs-only diff. -- 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]
