Skip to content

Conversation

@yiguolei
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

@Thearas
Copy link
Contributor

Thearas commented Jun 19, 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?

@yiguolei
Copy link
Contributor Author

run buildall

morningman
morningman previously approved these changes Jun 19, 2025
@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Jun 19, 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.

@github-actions github-actions bot removed the approved Indicates a PR has been approved by one committer. label Jun 19, 2025
@yiguolei
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17685	5196	5068	5068
q2	1958	300	200	200
q3	10364	1325	726	726
q4	10241	1032	548	548
q5	7624	2310	2339	2310
q6	183	164	137	137
q7	903	747	604	604
q8	9310	1301	1084	1084
q9	6840	5128	5112	5112
q10	6895	2409	1966	1966
q11	495	297	303	297
q12	358	366	237	237
q13	17775	3713	3117	3117
q14	237	241	224	224
q15	563	477	480	477
q16	454	453	380	380
q17	650	868	425	425
q18	7648	7365	7192	7192
q19	2378	997	562	562
q20	358	354	237	237
q21	4143	3340	2471	2471
q22	1103	1026	966	966
Total cold run time: 108165 ms
Total hot run time: 34340 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5267	5090	5113	5090
q2	242	330	235	235
q3	2207	2663	2325	2325
q4	1384	1794	1391	1391
q5	4226	4387	4481	4387
q6	219	169	128	128
q7	1964	1903	1771	1771
q8	2553	2525	2513	2513
q9	7212	7168	7264	7168
q10	3124	3243	2836	2836
q11	569	509	512	509
q12	715	789	626	626
q13	3495	4008	3336	3336
q14	290	297	283	283
q15	511	479	477	477
q16	442	483	434	434
q17	1171	1536	1380	1380
q18	7518	7180	7220	7180
q19	810	768	896	768
q20	1933	1979	1853	1853
q21	4833	4425	4361	4361
q22	1069	1032	985	985
Total cold run time: 51754 ms
Total hot run time: 50036 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 188888 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 9eb660a2fe45110641e31cd035d72e424f212e8a, data reload: false

query1	1016	393	400	393
query2	6528	1908	1890	1890
query3	6745	236	230	230
query4	26515	23790	23139	23139
query5	4344	638	484	484
query6	324	215	225	215
query7	4629	514	313	313
query8	277	234	220	220
query9	8646	2799	2821	2799
query10	458	349	267	267
query11	15884	15125	14892	14892
query12	166	110	104	104
query13	1656	561	417	417
query14	9767	6291	6396	6291
query15	209	194	171	171
query16	7591	642	501	501
query17	1190	726	604	604
query18	2016	422	331	331
query19	210	193	190	190
query20	126	126	121	121
query21	217	130	111	111
query22	4309	4283	4235	4235
query23	34578	33424	33493	33424
query24	8490	2417	2442	2417
query25	578	495	497	495
query26	1223	272	161	161
query27	2765	526	361	361
query28	4286	2302	2266	2266
query29	752	570	434	434
query30	281	226	206	206
query31	957	880	784	784
query32	78	63	68	63
query33	579	363	327	327
query34	827	876	573	573
query35	809	823	788	788
query36	999	1009	898	898
query37	116	101	78	78
query38	4149	4150	4068	4068
query39	1498	1434	1427	1427
query40	218	123	110	110
query41	62	60	59	59
query42	132	114	114	114
query43	506	535	502	502
query44	1382	889	865	865
query45	183	180	174	174
query46	877	1035	647	647
query47	1793	1816	1780	1780
query48	395	434	344	344
query49	738	529	413	413
query50	679	685	418	418
query51	4238	4188	4159	4159
query52	117	115	101	101
query53	241	254	189	189
query54	602	596	536	536
query55	96	90	92	90
query56	321	315	332	315
query57	1173	1247	1133	1133
query58	279	279	273	273
query59	2711	2704	2622	2622
query60	339	334	329	329
query61	131	127	124	124
query62	817	745	674	674
query63	239	200	195	195
query64	4293	1019	691	691
query65	4332	4236	4200	4200
query66	1090	413	318	318
query67	16052	15959	15798	15798
query68	8344	962	558	558
query69	476	334	324	324
query70	1220	1143	1162	1143
query71	481	341	329	329
query72	5704	4738	4765	4738
query73	749	651	375	375
query74	8939	9283	8780	8780
query75	3843	3243	2758	2758
query76	3673	1221	782	782
query77	793	399	306	306
query78	10009	10206	9366	9366
query79	2306	845	595	595
query80	642	543	459	459
query81	470	273	229	229
query82	433	134	100	100
query83	299	265	241	241
query84	294	115	91	91
query85	799	359	405	359
query86	341	318	322	318
query87	4479	4582	4403	4403
query88	3050	2417	2405	2405
query89	395	331	301	301
query90	1959	220	213	213
query91	146	146	115	115
query92	78	64	60	60
query93	1272	968	618	618
query94	678	418	312	312
query95	388	307	294	294
query96	529	606	295	295
query97	2758	2754	2682	2682
query98	243	225	209	209
query99	1506	1396	1269	1269
Total cold run time: 277120 ms
Total hot run time: 188888 ms

@doris-robot
Copy link

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

query1	0.04	0.03	0.03
query2	0.07	0.04	0.04
query3	0.23	0.07	0.06
query4	1.62	0.10	0.11
query5	0.44	0.42	0.43
query6	1.17	0.67	0.67
query7	0.02	0.02	0.02
query8	0.05	0.04	0.04
query9	0.60	0.52	0.53
query10	0.59	0.60	0.61
query11	0.16	0.11	0.11
query12	0.16	0.13	0.12
query13	0.63	0.61	0.61
query14	0.82	0.83	0.83
query15	0.90	0.88	0.88
query16	0.40	0.42	0.43
query17	1.12	1.08	1.12
query18	0.24	0.22	0.22
query19	2.00	1.87	1.93
query20	0.01	0.01	0.01
query21	15.39	0.96	0.58
query22	0.78	1.25	0.64
query23	14.84	1.40	0.62
query24	7.11	0.92	1.39
query25	0.51	0.10	0.20
query26	0.59	0.17	0.14
query27	0.06	0.06	0.04
query28	9.68	0.89	0.44
query29	12.54	4.02	3.36
query30	0.25	0.10	0.07
query31	2.83	0.61	0.40
query32	3.23	0.57	0.47
query33	3.15	3.05	3.26
query34	16.06	5.36	4.80
query35	4.88	4.89	4.82
query36	0.67	0.52	0.49
query37	0.09	0.07	0.07
query38	0.06	0.04	0.04
query39	0.04	0.03	0.03
query40	0.17	0.16	0.15
query41	0.08	0.03	0.03
query42	0.03	0.02	0.02
query43	0.04	0.03	0.04
Total cold run time: 104.35 s
Total hot run time: 29.96 s

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

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

Copy link
Member

@eldenmoon eldenmoon left a comment

Choose a reason for hiding this comment

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

LGTM

@doris-robot
Copy link

BE UT Coverage Report

Increment line coverage 50.00% (1/2) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 56.36% (15055/26711)
Line Coverage 45.14% (134696/298416)
Region Coverage 44.27% (67732/152986)
Branch Coverage 38.86% (34760/89442)

@hello-stephen
Copy link
Contributor

BE Regression && UT Coverage Report

Increment line coverage 50.00% (1/2) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 61.20% (16087/26285)
Line Coverage 50.73% (151315/298253)
Region Coverage 48.05% (86480/179977)
Branch Coverage 41.55% (42489/102256)

@yiguolei yiguolei merged commit 19a571d into apache:master Jun 19, 2025
27 of 31 checks passed
yiguolei added a commit to yiguolei/incubator-doris that referenced this pull request Jun 20, 2025
yiguolei added a commit to yiguolei/incubator-doris that referenced this pull request Jun 20, 2025
yiguolei added a commit to yiguolei/incubator-doris that referenced this pull request Jun 20, 2025
morrySnow pushed a commit that referenced this pull request Jun 20, 2025
…t cache and fix memleak in arrow input stream (#52074)

pick #51786
pick #51929
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants