Skip to content

Conversation

@Hastyshell
Copy link
Collaborator

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

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 Sep 16, 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?

@Hastyshell
Copy link
Collaborator Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17617	5198	5026	5026
q2	2026	317	218	218
q3	10260	1309	724	724
q4	10228	1027	530	530
q5	7555	2471	2338	2338
q6	181	170	142	142
q7	979	777	633	633
q8	9359	1367	1162	1162
q9	6987	5139	5191	5139
q10	6896	2419	1999	1999
q11	479	294	273	273
q12	351	364	231	231
q13	17767	3704	3112	3112
q14	244	244	220	220
q15	570	494	480	480
q16	1012	1010	953	953
q17	609	854	363	363
q18	7569	7298	7107	7107
q19	1115	956	571	571
q20	362	363	236	236
q21	3711	2652	2372	2372
q22	1087	1056	989	989
Total cold run time: 106964 ms
Total hot run time: 34818 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5199	5097	5031	5031
q2	249	335	234	234
q3	2180	2656	2313	2313
q4	1364	1797	1353	1353
q5	4273	4568	4654	4568
q6	230	176	136	136
q7	2084	1986	1836	1836
q8	2664	2698	2628	2628
q9	7542	7468	7289	7289
q10	3222	3307	2918	2918
q11	604	525	520	520
q12	762	805	653	653
q13	3504	3943	3382	3382
q14	306	335	295	295
q15	541	480	476	476
q16	1149	1132	1078	1078
q17	1228	1564	1390	1390
q18	8198	7719	7509	7509
q19	843	808	807	807
q20	1982	2054	1906	1906
q21	4771	4418	4247	4247
q22	1067	1040	1028	1028
Total cold run time: 53962 ms
Total hot run time: 51597 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 188812 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 fcae8a0c55d424317f8fe6d34d375c10f2eae867, data reload: false

query1	1099	447	401	401
query2	6553	1767	1734	1734
query3	6760	224	220	220
query4	26335	23739	23411	23411
query5	4704	671	480	480
query6	337	237	220	220
query7	4687	506	301	301
query8	320	257	251	251
query9	8706	2610	2625	2610
query10	473	353	292	292
query11	15581	15076	14838	14838
query12	180	129	115	115
query13	1664	555	431	431
query14	11212	9317	9259	9259
query15	212	200	168	168
query16	7710	691	502	502
query17	1199	776	638	638
query18	2048	425	340	340
query19	211	203	167	167
query20	131	125	123	123
query21	214	136	116	116
query22	4042	4103	3978	3978
query23	33948	33040	33064	33040
query24	8517	2459	2473	2459
query25	598	543	481	481
query26	1245	284	168	168
query27	2736	527	354	354
query28	4365	2256	2230	2230
query29	812	637	514	514
query30	295	233	202	202
query31	943	803	739	739
query32	92	80	77	77
query33	603	388	353	353
query34	800	856	531	531
query35	840	840	840	840
query36	998	1032	909	909
query37	121	116	84	84
query38	3517	3542	3521	3521
query39	1480	1422	1428	1422
query40	216	131	120	120
query41	65	64	68	64
query42	132	113	113	113
query43	495	524	476	476
query44	1364	871	835	835
query45	193	178	172	172
query46	866	1026	656	656
query47	1808	1838	1734	1734
query48	399	435	310	310
query49	813	501	407	407
query50	657	701	416	416
query51	3922	3958	3929	3929
query52	110	118	108	108
query53	235	282	190	190
query54	608	583	521	521
query55	87	89	84	84
query56	322	336	313	313
query57	1203	1196	1118	1118
query58	279	271	261	261
query59	2601	2673	2606	2606
query60	344	339	343	339
query61	163	159	161	159
query62	813	767	667	667
query63	232	209	199	199
query64	4478	1155	830	830
query65	4072	4015	3972	3972
query66	1099	427	335	335
query67	15416	15194	14874	14874
query68	9399	981	596	596
query69	481	313	285	285
query70	1373	1297	1271	1271
query71	570	345	323	323
query72	6066	5096	5213	5096
query73	744	691	362	362
query74	9214	9207	8660	8660
query75	4478	3366	2850	2850
query76	3941	1308	778	778
query77	991	419	334	334
query78	9672	9922	8887	8887
query79	1371	870	590	590
query80	651	575	494	494
query81	484	267	226	226
query82	239	164	132	132
query83	301	267	245	245
query84	310	112	90	90
query85	853	463	419	419
query86	336	320	297	297
query87	3798	3798	3690	3690
query88	2900	2217	2235	2217
query89	384	334	301	301
query90	2084	222	225	222
query91	169	167	137	137
query92	79	70	64	64
query93	1128	999	636	636
query94	685	452	307	307
query95	433	321	319	319
query96	483	568	282	282
query97	2938	2989	2868	2868
query98	234	229	216	216
query99	1442	1395	1291	1291
Total cold run time: 277511 ms
Total hot run time: 188812 ms

@doris-robot
Copy link

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

query1	0.06	0.05	0.05
query2	0.09	0.05	0.05
query3	0.26	0.08	0.08
query4	1.60	0.11	0.11
query5	0.27	0.27	0.24
query6	1.18	0.65	0.65
query7	0.04	0.03	0.03
query8	0.05	0.05	0.05
query9	0.63	0.52	0.52
query10	0.58	0.58	0.57
query11	0.16	0.12	0.12
query12	0.15	0.12	0.12
query13	0.67	0.62	0.62
query14	1.03	1.02	1.04
query15	0.86	0.84	0.88
query16	0.41	0.41	0.40
query17	1.05	1.06	1.08
query18	0.23	0.20	0.21
query19	1.94	1.79	1.80
query20	0.02	0.01	0.01
query21	15.46	0.94	0.60
query22	0.78	1.28	0.67
query23	14.87	1.40	0.60
query24	6.91	1.84	1.21
query25	0.49	0.22	0.06
query26	0.53	0.16	0.13
query27	0.07	0.05	0.05
query28	9.91	0.85	0.45
query29	12.65	3.91	3.27
query30	0.28	0.16	0.11
query31	2.83	0.59	0.39
query32	3.26	0.56	0.48
query33	3.05	3.12	3.09
query34	16.17	5.45	4.85
query35	4.99	4.95	4.92
query36	0.71	0.51	0.50
query37	0.10	0.07	0.08
query38	0.07	0.05	0.05
query39	0.04	0.02	0.03
query40	0.19	0.15	0.14
query41	0.08	0.03	0.04
query42	0.04	0.04	0.02
query43	0.05	0.03	0.03
Total cold run time: 104.81 s
Total hot run time: 30.27 s

@doris-robot
Copy link

BE UT Coverage Report

Increment line coverage 100.00% (1/1) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 52.33% (17542/33524)
Line Coverage 37.46% (158985/424453)
Region Coverage 32.04% (121239/378408)
Branch Coverage 33.37% (53161/159285)

@hello-stephen
Copy link
Contributor

BE Regression && UT Coverage Report

Increment line coverage 100.00% (1/1) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 71.03% (23279/32773)
Line Coverage 57.32% (242713/423400)
Region Coverage 52.78% (202354/383373)
Branch Coverage 54.34% (86938/159982)

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 github-actions bot added the approved Indicates a PR has been approved by one committer. label Sep 16, 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.

@dataroaring dataroaring merged commit d18a487 into apache:master Sep 16, 2025
28 of 31 checks passed
@morrySnow morrySnow mentioned this pull request Sep 22, 2025
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. dev/3.1.1-merged reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants