eric-maynard commented on issue #463: URL: https://github.com/apache/polaris/issues/463#issuecomment-2719229054
It's a good question. Ideally, any interface changes can be either minimal or purely additive (i.e. CallContext is added to interfaces, nothing is removed) so that the fix can be focused on removing threadlocal rather than also removing CallContext. Now that I am more familiar with Quarkus, I also feel that CallContext needs a refactor and that we have a few too many disjoint beans managed by CDI. But I would suggest that we decouple this work from the ThreadLocal work to keep the ThreadLocal removal as uncontroversial and as unlikely to disrupt existing deployments as possible. -- 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]
