Skip to content

Conversation

@Yukang-Lian
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 Jul 14, 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?

@Yukang-Lian
Copy link
Collaborator Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17616	5184	5031	5031
q2	1932	290	200	200
q3	10290	1297	729	729
q4	10224	1007	529	529
q5	7710	2458	2361	2361
q6	186	163	128	128
q7	932	752	607	607
q8	9322	1339	1096	1096
q9	6821	5131	5092	5092
q10	6900	2357	1963	1963
q11	467	290	269	269
q12	350	346	221	221
q13	17775	3681	3051	3051
q14	229	219	212	212
q15	554	496	476	476
q16	423	422	378	378
q17	605	895	361	361
q18	7601	7288	7158	7158
q19	1298	956	547	547
q20	338	325	214	214
q21	3681	3235	2299	2299
q22	359	321	298	298
Total cold run time: 105613 ms
Total hot run time: 33220 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5125	5059	5099	5059
q2	245	332	222	222
q3	2214	2709	2310	2310
q4	1349	1776	1402	1402
q5	4259	4357	4614	4357
q6	215	171	134	134
q7	2016	2019	1847	1847
q8	2654	2549	2713	2549
q9	7363	7258	7360	7258
q10	3141	3322	2925	2925
q11	597	493	483	483
q12	700	801	634	634
q13	3588	4013	3390	3390
q14	305	288	272	272
q15	532	490	483	483
q16	470	489	446	446
q17	1181	1599	1357	1357
q18	8155	7650	7737	7650
q19	814	842	847	842
q20	1916	1942	1844	1844
q21	4668	4294	4280	4280
q22	617	576	559	559
Total cold run time: 52124 ms
Total hot run time: 50303 ms

@doris-robot
Copy link

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

query1	1001	396	391	391
query2	6545	1736	1668	1668
query3	6742	212	217	212
query4	26292	23760	23491	23491
query5	4334	583	437	437
query6	287	207	188	188
query7	4627	509	296	296
query8	263	228	197	197
query9	8613	2597	2622	2597
query10	496	316	274	274
query11	15354	15245	14879	14879
query12	158	114	100	100
query13	1629	544	383	383
query14	8523	5887	5940	5887
query15	200	196	174	174
query16	7130	646	464	464
query17	946	722	605	605
query18	1998	426	337	337
query19	195	196	173	173
query20	127	118	112	112
query21	224	128	107	107
query22	4198	4134	4177	4134
query23	34138	33359	33060	33060
query24	8431	2430	2383	2383
query25	574	510	425	425
query26	1239	275	156	156
query27	2749	510	347	347
query28	4315	2134	2162	2134
query29	760	564	443	443
query30	281	219	193	193
query31	899	853	756	756
query32	72	64	61	61
query33	555	350	315	315
query34	807	838	526	526
query35	777	840	752	752
query36	991	1003	941	941
query37	116	100	80	80
query38	4285	4154	4077	4077
query39	1488	1600	1428	1428
query40	211	119	105	105
query41	60	53	51	51
query42	126	106	105	105
query43	510	516	489	489
query44	1304	824	831	824
query45	175	171	164	164
query46	843	1036	615	615
query47	1774	1785	1765	1765
query48	376	414	305	305
query49	758	486	390	390
query50	638	706	414	414
query51	5440	5584	5501	5501
query52	106	102	98	98
query53	231	263	186	186
query54	603	559	499	499
query55	82	82	86	82
query56	305	292	302	292
query57	1180	1209	1128	1128
query58	262	256	267	256
query59	2757	2664	2448	2448
query60	340	324	325	324
query61	134	122	125	122
query62	786	733	654	654
query63	221	189	196	189
query64	4370	985	647	647
query65	4277	4229	4165	4165
query66	1128	420	330	330
query67	15851	15659	15394	15394
query68	8451	896	531	531
query69	482	304	281	281
query70	1248	1125	1150	1125
query71	466	328	316	316
query72	5704	4757	4923	4757
query73	753	665	350	350
query74	9154	9052	8829	8829
query75	3814	3192	2739	2739
query76	3633	1148	704	704
query77	799	413	295	295
query78	10007	10288	9344	9344
query79	2137	853	596	596
query80	666	505	446	446
query81	488	269	224	224
query82	449	132	95	95
query83	263	259	228	228
query84	355	110	83	83
query85	777	361	317	317
query86	393	317	280	280
query87	4531	4406	4339	4339
query88	3449	2302	2299	2299
query89	388	325	291	291
query90	1840	219	214	214
query91	142	139	115	115
query92	81	59	57	57
query93	1636	974	574	574
query94	674	414	300	300
query95	399	303	306	303
query96	501	586	280	280
query97	2687	2753	2701	2701
query98	236	217	203	203
query99	1409	1378	1241	1241
Total cold run time: 275219 ms
Total hot run time: 187182 ms

@doris-robot
Copy link

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

query1	0.04	0.04	0.04
query2	0.08	0.04	0.04
query3	0.25	0.08	0.08
query4	1.62	0.10	0.10
query5	0.43	0.41	0.42
query6	1.16	0.68	0.66
query7	0.03	0.02	0.02
query8	0.06	0.04	0.04
query9	0.60	0.51	0.52
query10	0.57	0.57	0.56
query11	0.16	0.11	0.12
query12	0.16	0.11	0.11
query13	0.63	0.61	0.62
query14	0.79	0.82	0.83
query15	0.89	0.91	0.86
query16	0.38	0.38	0.39
query17	1.04	1.09	1.04
query18	0.23	0.21	0.21
query19	1.98	1.85	1.86
query20	0.01	0.01	0.02
query21	15.39	0.92	0.55
query22	0.75	1.12	0.64
query23	15.05	1.36	0.61
query24	7.03	1.06	0.54
query25	0.47	0.26	0.09
query26	0.58	0.16	0.15
query27	0.08	0.05	0.06
query28	9.50	0.95	0.43
query29	12.52	4.00	3.33
query30	0.25	0.10	0.06
query31	2.84	0.61	0.39
query32	3.22	0.58	0.47
query33	3.13	3.19	3.21
query34	15.95	5.32	4.74
query35	4.81	4.83	4.87
query36	0.69	0.52	0.48
query37	0.09	0.07	0.07
query38	0.06	0.04	0.04
query39	0.03	0.03	0.03
query40	0.17	0.14	0.13
query41	0.08	0.02	0.03
query42	0.03	0.03	0.02
query43	0.04	0.03	0.03
Total cold run time: 103.87 s
Total hot run time: 29.35 s

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 Jul 14, 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.

@doris-robot
Copy link

BE UT Coverage Report

Increment line coverage 0.00% (0/5) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 57.66% (15696/27222)
Line Coverage 46.36% (140008/302000)
Region Coverage 35.61% (104544/293554)
Branch Coverage 38.18% (46100/120742)

@gavinchou gavinchou merged commit 1d8701f into apache:master Jul 15, 2025
29 of 32 checks passed
dataroaring pushed a commit that referenced this pull request Jul 20, 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.0.7-merged dev/3.1.0-merged reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants