Skip to content

Conversation

@yujun777
Copy link
Contributor

@yujun777 yujun777 commented Aug 27, 2025

What problem does this PR solve?

related PR: #54414

Release note

None

Check List (For Author)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No code files have been changed.
      • Other reason
  • Behavior changed:

    • No.
    • Yes.
  • Does this need documentation?

    • No.
    • Yes.

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

@Thearas
Copy link
Contributor

Thearas commented Aug 27, 2025

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@yujun777
Copy link
Contributor Author

run buildall

@yujun777
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17594	5215	5047	5047
q2	2004	334	203	203
q3	10247	1275	711	711
q4	10223	1015	515	515
q5	7567	2397	2339	2339
q6	185	170	138	138
q7	938	762	627	627
q8	9351	1383	1085	1085
q9	6987	5046	5136	5046
q10	6971	2386	1993	1993
q11	499	294	274	274
q12	370	348	218	218
q13	17811	3657	2994	2994
q14	249	234	221	221
q15	582	511	499	499
q16	435	440	389	389
q17	594	845	370	370
q18	7374	7112	7078	7078
q19	1344	979	562	562
q20	353	343	224	224
q21	3706	3179	2368	2368
q22	1060	1038	980	980
Total cold run time: 106444 ms
Total hot run time: 33881 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5179	5103	5096	5096
q2	247	334	231	231
q3	2174	2671	2296	2296
q4	1381	1825	1355	1355
q5	4208	4298	4514	4298
q6	271	192	133	133
q7	2020	2021	1894	1894
q8	2672	2504	2831	2504
q9	7326	7343	7287	7287
q10	3124	3396	2833	2833
q11	603	523	521	521
q12	725	767	669	669
q13	3509	3988	3340	3340
q14	288	313	292	292
q15	521	467	467	467
q16	451	489	480	480
q17	1328	1463	1382	1382
q18	7724	7803	7598	7598
q19	791	903	964	903
q20	2025	1998	1878	1878
q21	4999	4496	4367	4367
q22	1098	1015	996	996
Total cold run time: 52664 ms
Total hot run time: 50820 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 185920 ms
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
TPC-DS sf100 test result on commit ca7a055b18e509de78b71be0df0128846cb1cafe, data reload: false

query1	1060	463	397	397
query2	6588	1757	1787	1757
query3	6755	239	219	219
query4	26119	23818	22887	22887
query5	4408	620	512	512
query6	347	248	233	233
query7	4644	506	299	299
query8	311	274	248	248
query9	8659	2903	2839	2839
query10	494	342	300	300
query11	15979	14984	14741	14741
query12	186	123	118	118
query13	1707	569	450	450
query14	9446	5867	5807	5807
query15	221	196	176	176
query16	7635	634	499	499
query17	1259	773	634	634
query18	2057	436	400	400
query19	203	194	172	172
query20	140	126	125	125
query21	217	124	111	111
query22	4052	4122	4081	4081
query23	33721	32757	33005	32757
query24	8027	2348	2414	2348
query25	576	513	431	431
query26	1237	272	171	171
query27	2703	520	345	345
query28	4381	2223	2187	2187
query29	743	627	505	505
query30	285	227	198	198
query31	915	792	729	729
query32	88	79	80	79
query33	568	382	354	354
query34	769	835	516	516
query35	819	847	786	786
query36	995	1017	924	924
query37	126	108	93	93
query38	4077	4061	4037	4037
query39	1478	1457	1435	1435
query40	220	130	124	124
query41	73	67	64	64
query42	133	113	118	113
query43	536	516	471	471
query44	1330	861	850	850
query45	185	176	170	170
query46	876	1000	642	642
query47	1779	1807	1730	1730
query48	399	428	333	333
query49	783	509	411	411
query50	641	679	394	394
query51	4091	4099	4156	4099
query52	117	112	106	106
query53	250	257	200	200
query54	625	585	551	551
query55	95	89	93	89
query56	333	349	314	314
query57	1222	1180	1130	1130
query58	299	284	288	284
query59	2658	2744	2741	2741
query60	367	356	352	352
query61	167	159	160	159
query62	790	728	679	679
query63	231	192	195	192
query64	4428	1183	830	830
query65	4304	4217	4213	4213
query66	1114	438	348	348
query67	15472	15284	15044	15044
query68	8709	912	584	584
query69	483	324	295	295
query70	1190	1119	1149	1119
query71	477	345	339	339
query72	5923	4950	4973	4950
query73	735	593	359	359
query74	9178	9107	8899	8899
query75	3977	3201	2613	2613
query76	3612	1126	721	721
query77	843	413	349	349
query78	9519	9699	8858	8858
query79	2447	801	596	596
query80	679	586	512	512
query81	484	260	230	230
query82	422	139	178	139
query83	296	268	248	248
query84	297	105	105	105
query85	887	468	437	437
query86	356	329	305	305
query87	4310	4295	4193	4193
query88	2994	2198	2187	2187
query89	394	328	291	291
query90	1953	228	223	223
query91	162	164	135	135
query92	89	80	71	71
query93	1119	966	646	646
query94	698	420	316	316
query95	416	330	324	324
query96	479	582	277	277
query97	2648	2693	2561	2561
query98	251	215	211	211
query99	1459	1409	1294	1294
Total cold run time: 274999 ms
Total hot run time: 185920 ms

@doris-robot
Copy link

ClickBench: Total hot run time: 32.79 s
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit ca7a055b18e509de78b71be0df0128846cb1cafe, data reload: false

query1	0.05	0.05	0.04
query2	0.09	0.05	0.05
query3	0.25	0.08	0.09
query4	1.61	0.12	0.12
query5	0.45	0.42	0.42
query6	1.17	0.65	0.66
query7	0.03	0.03	0.03
query8	0.05	0.04	0.05
query9	0.61	0.53	0.52
query10	0.59	0.57	0.55
query11	0.17	0.12	0.11
query12	0.15	0.13	0.12
query13	0.64	0.62	0.62
query14	0.80	0.86	0.84
query15	0.88	0.86	0.86
query16	0.39	0.40	0.41
query17	1.05	1.06	1.07
query18	0.22	0.20	0.19
query19	1.98	1.88	1.81
query20	0.02	0.01	0.01
query21	15.40	0.93	0.59
query22	0.76	1.23	0.73
query23	14.83	1.36	0.63
query24	6.61	1.52	0.74
query25	0.52	0.18	0.18
query26	0.57	0.16	0.13
query27	0.06	0.07	0.06
query28	9.77	0.89	0.43
query29	12.58	3.91	3.24
query30	3.12	3.09	3.02
query31	2.82	0.59	0.38
query32	3.24	0.55	0.48
query33	3.16	3.06	3.17
query34	16.00	5.50	4.83
query35	4.93	4.91	4.96
query36	0.70	0.54	0.49
query37	0.10	0.07	0.07
query38	0.05	0.06	0.04
query39	0.04	0.03	0.03
query40	0.18	0.15	0.14
query41	0.08	0.03	0.03
query42	0.04	0.03	0.03
query43	0.04	0.03	0.04
Total cold run time: 106.8 s
Total hot run time: 32.79 s

@hello-stephen
Copy link
Contributor

FE UT Coverage Report

Increment line coverage `` 🎉
Increment coverage report
Complete coverage report

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

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

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@yujun777
Copy link
Contributor Author

run p0

@yujun777
Copy link
Contributor Author

run external

@yujun777
Copy link
Contributor Author

run cloud_p0

@yujun777
Copy link
Contributor Author

run nonConcurrent

@yujun777
Copy link
Contributor Author

run p0

@yujun777
Copy link
Contributor Author

run cloud_p0

1 similar comment
@yujun777
Copy link
Contributor Author

run cloud_p0

@yujun777
Copy link
Contributor Author

run p0

Copy link
Contributor

@MoanasDaddyXu MoanasDaddyXu left a comment

Choose a reason for hiding this comment

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

LGTM

@hello-stephen
Copy link
Contributor

skip buildall

@hello-stephen hello-stephen merged commit 3554d3a into apache:master Aug 28, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants