Skip to content

Conversation

@Gabriel39
Copy link
Contributor

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

@hello-stephen
Copy link
Contributor

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?

@Gabriel39
Copy link
Contributor Author

run buildall

yiguolei
yiguolei previously approved these changes Sep 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2025

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

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels Sep 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2025

PR approved by anyone and no changes requested.

@Gabriel39
Copy link
Contributor Author

run buildall

@github-actions github-actions bot removed the approved Indicates a PR has been approved by one committer. label Sep 1, 2025
@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17618	5265	5038	5038
q2	2032	328	208	208
q3	10245	1249	693	693
q4	10215	1030	530	530
q5	7545	2368	2331	2331
q6	179	169	144	144
q7	949	744	647	647
q8	9364	1345	1073	1073
q9	6991	5151	5135	5135
q10	6935	2376	1939	1939
q11	492	312	288	288
q12	350	364	227	227
q13	17769	3668	3033	3033
q14	239	240	220	220
q15	573	509	481	481
q16	440	430	370	370
q17	593	849	360	360
q18	7570	7185	7161	7161
q19	1220	953	565	565
q20	336	341	250	250
q21	3731	2517	2331	2331
q22	1081	1013	990	990
Total cold run time: 106467 ms
Total hot run time: 34014 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5170	5210	5100	5100
q2	251	340	228	228
q3	2128	2674	2287	2287
q4	1310	1784	1372	1372
q5	4179	4194	4650	4194
q6	215	169	134	134
q7	2044	1950	1890	1890
q8	2687	2690	2586	2586
q9	7409	7581	7307	7307
q10	3074	3383	2845	2845
q11	576	519	517	517
q12	703	808	645	645
q13	3476	3976	3298	3298
q14	284	312	303	303
q15	518	486	467	467
q16	438	501	446	446
q17	1212	1569	1380	1380
q18	8070	7517	7586	7517
q19	805	825	877	825
q20	2050	2058	1950	1950
q21	4822	4414	4310	4310
q22	1101	1052	1002	1002
Total cold run time: 52522 ms
Total hot run time: 50603 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 186633 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 12de5101a4dfae6fe35133ecdbc307a2c03f25c8, data reload: false

query1	1071	435	403	403
query2	6551	1784	1756	1756
query3	6774	239	226	226
query4	26568	23576	23006	23006
query5	4402	665	499	499
query6	339	245	217	217
query7	4647	522	302	302
query8	314	262	251	251
query9	8617	2902	2992	2902
query10	584	337	289	289
query11	15849	15069	14736	14736
query12	184	123	127	123
query13	1691	566	449	449
query14	9634	5826	5841	5826
query15	216	192	177	177
query16	7665	636	515	515
query17	1217	733	625	625
query18	2034	424	334	334
query19	199	189	229	189
query20	139	137	128	128
query21	220	131	116	116
query22	4175	4191	4030	4030
query23	33817	33260	33008	33008
query24	8154	2376	2388	2376
query25	591	513	440	440
query26	1247	276	173	173
query27	2724	517	358	358
query28	4296	2271	2238	2238
query29	776	606	477	477
query30	299	249	189	189
query31	906	821	722	722
query32	91	80	79	79
query33	582	395	361	361
query34	795	862	516	516
query35	814	823	763	763
query36	989	1051	937	937
query37	127	118	95	95
query38	4047	4023	4015	4015
query39	1521	1437	1427	1427
query40	218	136	132	132
query41	74	63	60	60
query42	134	117	113	113
query43	515	529	472	472
query44	1343	865	874	865
query45	199	194	177	177
query46	862	1029	665	665
query47	1823	1827	1747	1747
query48	392	422	334	334
query49	764	503	410	410
query50	664	677	408	408
query51	4179	4109	4055	4055
query52	118	113	100	100
query53	263	263	204	204
query54	614	602	551	551
query55	101	94	92	92
query56	346	334	332	332
query57	1205	1215	1146	1146
query58	295	285	287	285
query59	2671	2806	2705	2705
query60	370	363	349	349
query61	166	153	150	150
query62	816	722	679	679
query63	247	197	195	195
query64	4436	1133	825	825
query65	4351	4241	4232	4232
query66	1103	450	357	357
query67	15227	15038	15122	15038
query68	8113	927	591	591
query69	480	333	298	298
query70	1267	1141	1168	1141
query71	495	356	335	335
query72	5753	5027	5162	5027
query73	724	652	364	364
query74	9274	9294	8861	8861
query75	3829	3103	2630	2630
query76	3709	1151	760	760
query77	801	427	348	348
query78	9653	9779	8913	8913
query79	2265	897	593	593
query80	750	581	520	520
query81	478	263	228	228
query82	441	144	115	115
query83	284	260	250	250
query84	314	117	99	99
query85	882	465	422	422
query86	347	309	316	309
query87	4263	4284	4192	4192
query88	3391	2296	2254	2254
query89	399	341	293	293
query90	1953	232	234	232
query91	161	159	135	135
query92	93	79	72	72
query93	1690	1010	655	655
query94	693	417	320	320
query95	428	338	336	336
query96	493	590	282	282
query97	2681	2735	2590	2590
query98	255	216	222	216
query99	1437	1442	1301	1301
Total cold run time: 276158 ms
Total hot run time: 186633 ms

@doris-robot
Copy link

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

query1	0.06	0.04	0.05
query2	0.10	0.05	0.05
query3	0.25	0.09	0.09
query4	1.61	0.12	0.11
query5	0.46	0.42	0.41
query6	1.15	0.64	0.67
query7	0.02	0.03	0.03
query8	0.06	0.04	0.04
query9	0.60	0.56	0.53
query10	0.58	0.58	0.57
query11	0.18	0.12	0.11
query12	0.16	0.13	0.12
query13	0.62	0.63	0.62
query14	0.86	0.81	0.84
query15	0.88	0.86	0.89
query16	0.39	0.41	0.39
query17	1.03	1.06	1.07
query18	0.22	0.21	0.21
query19	1.98	1.90	1.84
query20	0.02	0.01	0.02
query21	15.41	0.94	0.58
query22	0.76	1.23	0.78
query23	14.80	1.39	0.66
query24	6.75	1.39	1.02
query25	0.51	0.30	0.08
query26	0.56	0.16	0.14
query27	0.07	0.06	0.05
query28	9.34	0.95	0.45
query29	12.60	3.92	3.21
query30	3.08	3.10	2.99
query31	2.83	0.60	0.38
query32	3.26	0.57	0.48
query33	3.13	3.04	3.09
query34	16.20	5.41	4.85
query35	5.01	4.94	4.93
query36	0.71	0.52	0.51
query37	0.11	0.08	0.07
query38	0.06	0.04	0.04
query39	0.04	0.03	0.04
query40	0.20	0.16	0.14
query41	0.08	0.04	0.04
query42	0.04	0.03	0.02
query43	0.04	0.04	0.04
Total cold run time: 106.82 s
Total hot run time: 33.07 s

@hello-stephen
Copy link
Contributor

BE UT Coverage Report

Increment line coverage 83.33% (5/6) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 51.65% (17121/33150)
Line Coverage 37.18% (156399/420670)
Region Coverage 31.83% (119224/374590)
Branch Coverage 33.19% (52425/157937)

@hello-stephen
Copy link
Contributor

BE Regression && UT Coverage Report

Increment line coverage 83.33% (5/6) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 70.56% (22977/32566)
Line Coverage 56.85% (239080/420542)
Region Coverage 52.25% (198609/380107)
Branch Coverage 53.91% (85676/158918)

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

github-actions bot commented Sep 2, 2025

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

@Gabriel39 Gabriel39 merged commit 6b4c72c into apache:master Sep 2, 2025
26 of 29 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