Uh oh!
There was an error while loading. Please reload this page.
[fix](workload policy)Remove set session variable workload policy action - #64856
Conversation
hello-stephen
commented
Jun 25, 2026
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
yiguolei
commented
Jun 25, 2026
Please also remove the document from master and 4.x |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Uh oh!
There was an error while loading. Please reload this page.
feiniaofeiafei
commented
Jun 26, 2026
run buildall |
hello-stephen
commented
Jun 26, 2026
TPC-H: Total hot run time: 29248 ms |
hello-stephen
commented
Jun 26, 2026
TPC-DS: Total hot run time: 171186 ms |
hello-stephen
commented
Jun 26, 2026
ClickBench: Total hot run time: 25.18 s |
hello-stephen
commented
Jun 26, 2026
FE UT Coverage ReportIncrement line coverage |
hello-stephen
commented
Jun 26, 2026
FE Regression Coverage ReportIncrement line coverage |
morrySnow
commented
Jun 30, 2026
add related PR |
PR approved by at least one committer and no changes requested. |
yiguolei
commented
Jul 6, 2026
/review |
### What problem does this PR solve? Issue Number: None Related PR: None Problem Summary: The workload policy set_session_variable action has problematic behavior and is being removed. This change removes its parser special case, runtime action implementation, and action type registration, then updates workload policy tests so new policies reject set_session_variable as an invalid action. Existing BE-side workload policy actions remain unchanged. ### Release note The set_session_variable workload policy action is no longer supported. ### Check List (For Author) - Test: - Unit Test: ./run-fe-ut.sh --run org.apache.doris.resource.workloadschedpolicy.WorkloadSchedPolicyMgrTest (passed; rerun with elevated permissions after Mockito inline attach failed under sandbox) - Checkstyle: mvn checkstyle:check -pl fe-core,fe-sql-parser (passed) - Regression test: ./run-regression-test.sh --run -d workload_manager_p0 -s test_workload_sched_policy (attempted, failed because local 127.0.0.1:9030 Doris cluster was not running) - Behavior changed: Yes. set_session_variable workload policy action is rejected. - Does this need documentation: No
4fc1b5e to
c5eaf8cComparefeiniaofeiafei
commented
Aug 19, 2026
run buildall |
hello-stephen
commented
Aug 19, 2026
TPC-H: Total hot run time: 17453 ms |
hello-stephen
commented
Aug 19, 2026
TPC-DS: Total hot run time: 83295 ms |
hello-stephen
commented
Aug 19, 2026
ClickBench: Total hot run time: 14.56 s |
304015afeiniaofeiafei
commented
Aug 20, 2026
run buildall |
hello-stephen
commented
Aug 20, 2026
TPC-H: Total hot run time: 17544 ms |
hello-stephen
commented
Aug 20, 2026
TPC-DS: Total hot run time: 83200 ms |
hello-stephen
commented
Aug 20, 2026
ClickBench: Total hot run time: 14.72 s |
hello-stephen
commented
Aug 20, 2026
FE UT Coverage ReportIncrement line coverage |
hello-stephen
commented
Aug 20, 2026
FE Regression Coverage ReportIncrement line coverage |
Uh oh!
There was an error while loading. Please reload this page.
…ion (#64856) ### What problem does this PR solve? Related PR: #28443 Problem Summary: The workload policy set_session_variable action has problematic behavior and is being removed. This change removes its parser special case, runtime action implementation, and action type registration, then updates workload policy tests so new policies reject set_session_variable as an invalid action. Existing BE-side workload policy actions remain unchanged. ### Release note The set_session_variable workload policy action is no longer supported.
What problem does this PR solve?
Related PR: #28443
Problem Summary: The workload policy set_session_variable action has problematic behavior and is being removed. This change removes its
parser special case, runtime action implementation, and action type registration, then updates workload policy tests so new policies reject set_session_variable as an invalid action. Existing BE-side workload policy actions remain unchanged.
Release note
The set_session_variable workload policy action is no longer supported.
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)