morrySnow commented on code in PR #67811:
URL: https://github.com/apache/doris/pull/67811#discussion_r4004494711
##########
fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java:
##########
@@ -457,7 +455,6 @@ public String toString() {
public static final String REWRITE_OR_TO_IN_PREDICATE_THRESHOLD =
"rewrite_or_to_in_predicate_threshold";
public static final String NEREIDS_CBO_PENALTY_FACTOR =
"nereids_cbo_penalty_factor";
- public static final String ENABLE_NEREIDS_TRACE = "enable_nereids_trace";
public static final String ENABLE_EXPR_TRACE = "enable_expr_trace";
Review Comment:
Fixed in ef2cd1599fcd. Added lowercase `enable_nereids_trace` and
`nereids_trace_event_mode` to `VariableMgr.REMOVED_SESSION_VAR_NAMES`. The
compatibility test covers case-insensitive lookup, direct and forwarded `SET`,
legacy `VariableExpr` reads, and Nereids literal reads; `VariableMgrTest`
passes 13/13.
--
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]
