dimas-b commented on code in PR #1817:
URL: https://github.com/apache/polaris/pull/1817#discussion_r2132573217


##########
quarkus/service/src/main/java/org/apache/polaris/service/quarkus/task/QuarkusTaskExecutorImpl.java:
##########
@@ -73,6 +79,7 @@ protected void handleTask(long taskEntityId, CallContext 
callContext, int attemp
             .setAttribute("polaris.task.attempt", attempt)
             .startSpan();
     try (Scope ignored = span.makeCurrent()) {
+      polarisRequestContext.setRealmContext(callContext.getRealmContext());

Review Comment:
   Eventually - yes. With the recent config changes this PR is no longer 
meaningful in isolation, but it shows how CDI can be done for tasks. 
   
   I also opened a bigger related proposal on the dev ML: 
https://lists.apache.org/thread/0cyrzft2oon28otxlmmhvd5671rd3r3d



-- 
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