gh-yzou commented on PR #1758:
URL: https://github.com/apache/polaris/pull/1758#issuecomment-2931846027

   @adutra Yes, we were trying to leverage CDI to  get rid of the thread local 
usage CallContext.getCurrentContext() in DefaultConfigurationStore, and didn't 
realize that it doesn't work with task executor.
   
   ```
   The proper way to pass the realm context here would be as a method 
parameter, for example by passing CallContext instead of PolarisCallContext
   ```
   We actually only need the realm, not even the whole call context. This PR 
currently introduces the new method, i will have follow up PRs to remove the 
other usages step by step.
   
   


-- 
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...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to