shauryachats commented on code in PR #17444:
URL: https://github.com/apache/pinot/pull/17444#discussion_r2654532869


##########
pinot-query-planner/src/main/java/org/apache/pinot/query/planner/physical/PinotDispatchPlanner.java:
##########
@@ -62,7 +63,7 @@ public DispatchableSubPlan createDispatchableSubPlan(SubPlan 
subPlan) {
     PlanFragment rootFragment = subPlan.getSubPlanRoot();
     PlanNode rootNode = rootFragment.getFragmentRoot();
     // 1. start by visiting the sub plan fragment root.
-    rootNode.visit(new DispatchablePlanVisitor(_tableCache), context);
+    rootNode.visit(new DispatchablePlanVisitor(_tableCache, routingContext), 
context);

Review Comment:
   Physical optimizer currently does not support logical tables at all. We 
should build support for it and multi-cluster routing subsequently in the next 
PR.



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to