adutra opened a new pull request, #12562:
URL: https://github.com/apache/iceberg/pull/12562
Currently, `AuthSessionCache` is not generic and only accepts keys of type
String.
This wasn't a good design choice. A common pattern I've seen already is to
create cache entries where the key is some arbitrary configuration object.
To allow this, we need to be able to create generic caches e.g.`new
AuthSessionCache<MyAuthConfig, MyAuthSession>`. This PR enables that.
Unfortunately that requires some revapi exceptions. I think they are
acceptable, but I could also introduce a second `AuthSessionCache` instead and
deprecate the existing one, if that's better.
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
