Skip to content

Conversation

@Gabriel39
Copy link
Contributor

@Gabriel39 Gabriel39 commented Jan 13, 2026

What problem does this PR solve?

Introduced by #59669

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 Jan 13, 2026

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

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17627	4233	4035	4035
q2	2049	340	235	235
q3	10182	1293	708	708
q4	10224	885	317	317
q5	7513	2097	1806	1806
q6	186	167	140	140
q7	899	767	653	653
q8	9260	1391	1170	1170
q9	4892	4674	4435	4435
q10	6776	1763	1419	1419
q11	511	296	280	280
q12	679	731	576	576
q13	17788	3765	3038	3038
q14	292	292	270	270
q15	586	516	511	511
q16	680	678	623	623
q17	638	747	532	532
q18	7043	6355	6211	6211
q19	1087	977	596	596
q20	373	341	243	243
q21	2918	2371	2160	2160
q22	1002	998	950	950
Total cold run time: 103205 ms
Total hot run time: 30908 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4130	4011	4026	4011
q2	338	409	307	307
q3	2069	2583	2232	2232
q4	1315	1719	1300	1300
q5	4036	3978	3969	3969
q6	206	169	130	130
q7	1866	1805	1677	1677
q8	2657	2570	2532	2532
q9	7181	7099	7039	7039
q10	2527	2787	2223	2223
q11	615	488	511	488
q12	715	744	628	628
q13	3585	4046	3382	3382
q14	312	309	277	277
q15	654	529	496	496
q16	657	697	642	642
q17	1150	1374	1359	1359
q18	8156	7611	7587	7587
q19	879	871	834	834
q20	2207	1998	1895	1895
q21	4978	4374	4309	4309
q22	1120	1111	1043	1043
Total cold run time: 51353 ms
Total hot run time: 48360 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 172648 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 1eb21a6594fec12f20095806ff553da55199121a, data reload: false

query5	4404	610	478	478
query6	315	220	205	205
query7	4225	453	259	259
query8	342	248	243	243
query9	8683	2874	2869	2869
query10	524	381	325	325
query11	15283	15108	14791	14791
query12	180	127	115	115
query13	1261	492	355	355
query14	5906	2947	2719	2719
query14_1	2613	2561	2601	2561
query15	194	189	170	170
query16	992	482	440	440
query17	1064	635	535	535
query18	2435	419	320	320
query19	225	220	185	185
query20	132	127	122	122
query21	223	136	118	118
query22	4032	4343	3981	3981
query23	15953	15543	15210	15210
query23_1	15350	15498	15388	15388
query24	7139	1549	1152	1152
query24_1	1193	1165	1182	1165
query25	569	501	416	416
query26	1257	266	156	156
query27	2766	441	272	272
query28	4587	2163	2143	2143
query29	774	549	439	439
query30	315	243	209	209
query31	801	634	517	517
query32	85	77	78	77
query33	542	350	300	300
query34	883	898	523	523
query35	726	776	659	659
query36	886	918	807	807
query37	129	94	90	90
query38	2817	2692	2687	2687
query39	785	749	727	727
query39_1	712	710	706	706
query40	219	136	119	119
query41	71	69	67	67
query42	109	106	108	106
query43	439	433	401	401
query44	1320	744	733	733
query45	190	189	174	174
query46	835	941	566	566
query47	1470	1441	1379	1379
query48	306	320	240	240
query49	631	441	347	347
query50	602	282	213	213
query51	3837	3777	3775	3775
query52	104	110	99	99
query53	310	326	278	278
query54	300	289	270	270
query55	80	79	74	74
query56	322	319	335	319
query57	1017	955	897	897
query58	297	268	263	263
query59	2068	2175	2082	2082
query60	338	380	313	313
query61	153	149	151	149
query62	382	362	312	312
query63	297	262	269	262
query64	4880	1277	974	974
query65	3738	3645	3780	3645
query66	1460	422	318	318
query67	15023	15640	14818	14818
query68	7278	979	695	695
query69	502	355	320	320
query70	1052	932	945	932
query71	358	309	280	280
query72	5724	3329	3426	3329
query73	762	714	298	298
query74	8785	8754	8675	8675
query75	2802	2788	2424	2424
query76	3349	1066	653	653
query77	512	372	294	294
query78	9679	9895	9162	9162
query79	1354	861	556	556
query80	611	581	487	487
query81	521	258	230	230
query82	208	155	116	116
query83	267	249	242	242
query84	257	115	94	94
query85	884	479	449	449
query86	400	310	324	310
query87	2901	2846	2742	2742
query88	2889	2108	2084	2084
query89	390	350	311	311
query90	2190	162	159	159
query91	167	158	140	140
query92	86	72	72	72
query93	1667	870	524	524
query94	583	325	298	298
query95	588	324	360	324
query96	559	476	193	193
query97	2330	2374	2299	2299
query98	208	205	205	205
query99	588	593	594	593
Total cold run time: 252809 ms
Total hot run time: 172648 ms

@doris-robot
Copy link

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

query1	0.05	0.05	0.05
query2	0.10	0.05	0.04
query3	0.26	0.09	0.09
query4	1.61	0.12	0.12
query5	0.28	0.25	0.24
query6	1.15	0.66	0.66
query7	0.03	0.03	0.03
query8	0.06	0.04	0.04
query9	0.57	0.51	0.51
query10	0.55	0.55	0.54
query11	0.14	0.09	0.10
query12	0.15	0.11	0.11
query13	0.61	0.60	0.58
query14	0.97	0.93	0.93
query15	0.79	0.77	0.78
query16	0.42	0.42	0.39
query17	0.99	1.02	1.01
query18	0.22	0.21	0.21
query19	1.92	1.83	1.84
query20	0.02	0.02	0.01
query21	15.43	0.28	0.14
query22	5.31	0.05	0.05
query23	16.21	0.27	0.11
query24	1.96	0.29	0.27
query25	0.12	0.07	0.07
query26	0.13	0.13	0.15
query27	0.08	0.06	0.11
query28	4.22	1.08	0.89
query29	12.55	3.92	3.18
query30	0.28	0.13	0.12
query31	2.82	0.61	0.38
query32	3.25	0.56	0.47
query33	3.01	3.12	3.02
query34	15.80	5.17	4.44
query35	4.42	4.48	4.45
query36	0.66	0.49	0.48
query37	0.10	0.07	0.07
query38	0.07	0.04	0.04
query39	0.04	0.02	0.03
query40	0.17	0.14	0.13
query41	0.09	0.03	0.04
query42	0.05	0.03	0.03
query43	0.04	0.03	0.04
Total cold run time: 97.7 s
Total hot run time: 26.86 s

@doris-robot
Copy link

BE UT Coverage Report

Increment line coverage 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 52.99% (18973/35803)
Line Coverage 39.04% (175833/450373)
Region Coverage 33.60% (136048/404850)
Branch Coverage 34.65% (58853/169837)

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

@hello-stephen
Copy link
Contributor

skip check_coverage

@Gabriel39 Gabriel39 merged commit 2e40e45 into apache:master Jan 14, 2026
28 of 30 checks passed
Gabriel39 added a commit to Gabriel39/incubator-doris that referenced this pull request Jan 15, 2026
@Gabriel39 Gabriel39 mentioned this pull request Jan 15, 2026
16 tasks
yiguolei pushed a commit that referenced this pull request Jan 16, 2026
### What problem does this PR solve?

pick #59488 #59669 #59733 #59831

### Check List (For Author)

- Test <!-- At least one of them must be included. -->
    - [ ] 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 <!-- Add your reason?  -->

- Behavior changed:
    - [ ] No.
    - [ ] Yes. <!-- Explain the behavior change -->

- Does this need documentation?
    - [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
apache/doris-website#1214 -->

### Check List (For Reviewer who merge this PR)

- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR
should merge into -->
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/4.0.3-merged reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants