[Improvement] Refactor Iceberg REST server credential code #6024
Assignees
Labels
Comments
jerryshao
pushed a commit
that referenced
this issue
Dec 30, 2024
…ential component in Gravitino server (#6021) ### What changes were proposed in this pull request? 1. reuse `CatalogCredentialManager` to manage the credentials in Iceberg catalog. 2. use `CatalogWrapperForREST` to manage some REST specific operations like credential vending. 3. depracate `catalog-provider-type` , use `catalog-providers` instead. ### Why are the changes needed? Fix: #6024 ### Does this PR introduce _any_ user-facing change? yes, depracate `catalog-provider-type` , use `catalog-providers`, do some compatibility work. ### How was this patch tested? run with s3 token
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What would you like to be improved?
After supporting credential vending in Gravitino server, we introduced some common component like
CatalogCredentialManagerCredentialCache, we 'd do a refactor for Iceberg REST server credential code to reuse the componentHow should we improve?
No response
The text was updated successfully, but these errors were encountered: