Skip to content

branch-3.0: [fix](fe) multi statements different behavior in master and follower (#52144) #52630

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
dataroaring merged 1 commit into from
Jul 7, 2025
Merged

branch-3.0: [fix](fe) multi statements different behavior in master and follower (#52144) #52630

dataroaring merged 1 commit into from
Jul 7, 2025

Conversation

@mymeiyi
Copy link
Contributor

@mymeiyi mymeiyi commented Jul 2, 2025

pick #52144

@mymeiyi mymeiyi requested a review from dataroaring as a code owner July 2, 2025 03:01
@mymeiyi
Copy link
Contributor Author

mymeiyi commented Jul 2, 2025

run buildall

@doris-robot
Copy link

doris-robot commented Jul 2, 2025

TPC-H: Total hot run time: 39792 ms 
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 16408349eccfd9a7e0bf4b05223512ff4c708207, data reload: false

------ Round 1 ----------------------------------
q1	17651	6773	6585	6585
q2	2075	171	167	167
q3	10781	1066	1160	1066
q4	10545	765	702	702
q5	7743	2932	2896	2896
q6	213	138	136	136
q7	990	615	588	588
q8	9367	2022	2041	2022
q9	6642	6420	6441	6420
q10	7002	2262	2314	2262
q11	449	255	254	254
q12	400	215	212	212
q13	17795	2991	2975	2975
q14	223	218	200	200
q15	508	469	489	469
q16	471	385	380	380
q17	971	546	619	546
q18	7148	6672	6764	6672
q19	1399	1094	1053	1053
q20	487	198	204	198
q21	3892	3191	3008	3008
q22	1109	990	981	981
Total cold run time: 107861 ms
Total hot run time: 39792 ms

----- Round 2, with runtime_filter_mode=off -----
q1	6561	6591	6538	6538
q2	331	230	226	226
q3	2951	2741	2833	2741
q4	2022	1766	1776	1766
q5	5845	5767	5781	5767
q6	210	130	131	130
q7	2215	1777	1807	1777
q8	3364	3557	3561	3557
q9	8885	8715	8954	8715
q10	3553	3528	3534	3528
q11	597	506	484	484
q12	793	609	580	580
q13	8243	3111	3206	3111
q14	304	264	271	264
q15	509	472	458	458
q16	489	440	438	438
q17	1860	1647	1596	1596
q18	8344	7925	7700	7700
q19	1699	1560	1531	1531
q20	2149	1771	1811	1771
q21	5144	4972	5057	4972
q22	1166	1066	1084	1066
Total cold run time: 67234 ms
Total hot run time: 58716 ms

dataroaring
dataroaring approved these changes Jul 7, 2025
View reviewed changes
Copy link
Contributor

@dataroaring dataroaring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2025

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Jul 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2025

PR approved by anyone and no changes requested.

@dataroaring dataroaring merged commit 9b0bdc8 into apache:branch-3.0 Jul 7, 2025
23 of 25 checks passed
@mymeiyi mymeiyi deleted the 3.0-pick-52144 branch September 10, 2025 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dataroaring dataroaring dataroaring approved these changes

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by one committer. reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mymeiyi @doris-robot @dataroaring