BiteTheDDDDt opened a new pull request, #64479:
URL: https://github.com/apache/doris/pull/64479

   ### What problem does this PR solve?
   
   Issue Number: None
   
   Related PR: #64285
   
   Problem Summary:
   
   `enable_segment_limit_pushdown` is the replacement switch for the old common 
expr pushdown session variables that were removed as no-op FE variables. The 
variable was already marked fuzzy, but fuzzy initialization still skipped 
randomizing it because many regression cases explicitly pinned the old 
variables.
   
   This PR migrates regression cases that set `enable_common_expr_pushdown` or 
`enable_common_expr_pushdown_for_inverted_index` to use 
`enable_segment_limit_pushdown` instead, including both true and false 
settings. Cases that had conflicting old settings in one `SET_VAR` are 
normalized to a single `enable_segment_limit_pushdown` setting. After the case 
cleanup, fuzzy mode now randomizes `enableSegmentLimitPushdown`.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test: Manual test
       - `git diff --check HEAD^ HEAD`
       - `git diff --cached --check`
       - `rg` verification that regression suites no longer reference the old 
common expr pushdown variables outside data files
       - `rg` verification that no regression hint has conflicting or duplicate 
`enable_segment_limit_pushdown` settings
   - Behavior changed: No
   - Does this need documentation: No
   


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