HappenLee commented on code in PR #68672:
URL: https://github.com/apache/doris/pull/68672#discussion_r4164416865
##########
gensrc/thrift/PlanNodes.thrift:
##########
@@ -1809,6 +1809,9 @@ struct TPlanNode {
106: optional list<i32> topn_filter_source_node_ids
107: optional i32 nereids_id
+ // FE expression eligibility, independent of the session switch. An old FE
has not checked
+ // volatility, so absence must disable condition cache on a new BE.
+ 108: optional bool enable_condition_cache = false
Review Comment:
This new-FE/old-BE combination does not occur in our supported upgrade
procedure: all BEs are upgraded before the FEs, as documented in
https://doris.apache.org/docs/4.x/admin-manual/cluster-management/upgrade/.
During the supported mixed-version phase (old FE/new BE), the optional flag
defaults to false and safely disables condition cache. We will retain this
compatibility design and address the nested-cast and runtime-filter findings
separately.
--
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]
