924060929 commented on code in PR #67177:
URL: https://github.com/apache/doris/pull/67177#discussion_r3955920390
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/NereidsPlanner.java:
##########
@@ -267,6 +270,7 @@ private Plan planWithLock(LogicalPlan plan,
PhysicalProperties requireProperties
// pre-process logical plan out of memo, e.g. process SET_VAR hint
plan = preprocess(plan);
+ freezeLocalShufflePlannerMode();
Review Comment:
Fixed in fb0f9f27106. The query-level execution version and effective
local-shuffle planner mode are now frozen immediately after analysis reapplies
retained SET_VAR hints, while still preceding rewrite and Cascades property
derivation. NereidsCoordinatorTest reuses the retained prepared plan after the
PREPARE-time session values are reverted and covers both directions: a retained
hint enables local-shuffle planning over a disabled base session, and a
retained hint disables it over an enabled base session. The final focused FE
suite passes 85/85.
--
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]
