szlta commented on PR #13225: URL: https://github.com/apache/iceberg/pull/13225#issuecomment-5205433977
Thanks for the detailed status description @singhpk234. The way I see it, right now we have these 2 PRs (#17155 and this) opened in the topic, and even though this PR is outside the scope of credential vending, there's a dependency: the spec proposal in its current form disallows hybrid credential consumption. This current REST catalog support implementation has no guardrails against that happening, but that said the spec is not yet merged either. I believe if we add the limitation of vended (storage) credential disablement to this PR, we can break this dependency and then this PR would clearly aim at the static credential case for both storage and KMS. Let me know what y'all think. -- 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]
