apache / polaris

Apache Polaris, the interoperable, open source catalog for Apache Iceberg

https://polaris.apache.org/

Apache License 2.0

1.13k stars 124 forks source link

DefaultContextResolver should respect the defaultRealm configuration #309

Closed eric-maynard closed 3 weeks ago

eric-maynard commented 1 month ago

Description

Previously DefaultContextResolver would always use default-realm as the default realm regardless of the configuration for defaultRealm. This PR fixes this behavior.

Fixes #281

Type of change

Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] Documentation update
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

Please delete options that are not relevant.

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

flyrain commented 3 weeks ago

Thanks @eric-maynard for the fix!