github-actions[bot] commented on code in PR #66640:
URL: https://github.com/apache/doris/pull/66640#discussion_r3773933088
##########
fe/fe-core/src/main/java/org/apache/doris/qe/VariableMgr.java:
##########
@@ -1087,6 +1087,9 @@ public static void forceUpdateVariables() {
VariableMgr.refreshDefaultSessionVariables(updateInfo,
SessionVariable.ENABLE_SQL_CACHE,
String.valueOf(true));
+ VariableMgr.refreshDefaultSessionVariables(updateInfo,
Review Comment:
This repair is under a migration fence that the released 4.1 line has
already consumed. A 3.x cluster carries `planner=false`; 4.1.0-4.1.3 then
persist that value with `variable_version=400`, and this patched build replays
`false/400` and skips the new assignment. Thus direct 3.x -> patched-4.1 works,
but the staged/maintenance upgrade remains on the legacy planner. Please
allocate a fresh one-shot version (for example 401), put this assignment under
`< 401`, persist 401 last, and make the regression start from 400
(acknowledging that this intentionally overrides indistinguishable explicit
`false` values). Otherwise the already-upgraded population is never repaired.
--
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]
