gh-yzou commented on code in PR #1817:
URL: https://github.com/apache/polaris/pull/1817#discussion_r2132542508
##########
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:
Actually, i think we probably need to propagate the whole callcontext,
instead of just the realmContext, because the whole callContext is needed for
the background executor, the realmContext is just what needed for
getConfiguration, which is called during the task execution.
--
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
