adutra opened a new pull request, #589: URL: https://github.com/apache/polaris/pull/589
⚠️ Not ready for review yet! Requires #469. Fixes #463. Removing `CallContext` is done in 2 steps: 1. Removal of `PolarisCallContext`: this component is an aggregation of 3 application-scoped beans, and one request-scoped bean (the meta store session). It is "easy" to replace it by the components that are actually needed, however since it's used nearly everywhere, the blast radius is quite big. Application scoped beans are generally replaced with constructor injection, and request-scoped ones are passed through additional method arguments. 2. Removal of `CallContext` proper. This is easier after step 1. The only issue is to revisit how `PolarisCatalogHandlerWrapper` closes the base catalog. The `doCatalogOperation` trick wasn't being done at the right level imho and could leave some catalogs unclosed in case of runtime errors. -- 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]
