Skip to content

Conversation

@seawinde
Copy link
Contributor

@seawinde seawinde commented Jul 2, 2025

cherry-pick part code from #40202
pr: #40202
commitId: 81f3c48

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

@seawinde seawinde requested a review from dataroaring as a code owner July 2, 2025 02:56
@Thearas
Copy link
Contributor

Thearas commented Jul 2, 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?

@seawinde
Copy link
Contributor Author

seawinde commented Jul 2, 2025

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17856	7059	6634	6634
q2	2075	174	162	162
q3	11063	1077	1154	1077
q4	10512	750	734	734
q5	7743	2932	2791	2791
q6	222	137	139	137
q7	977	615	602	602
q8	9353	1991	2033	1991
q9	6587	6409	6463	6409
q10	7006	2333	2282	2282
q11	468	271	266	266
q12	402	212	209	209
q13	17790	2967	3018	2967
q14	248	207	209	207
q15	510	490	465	465
q16	491	380	370	370
q17	992	592	556	556
q18	7473	6719	6797	6719
q19	1441	1098	1060	1060
q20	477	204	207	204
q21	3912	3132	3004	3004
q22	1100	1002	996	996
Total cold run time: 108698 ms
Total hot run time: 39842 ms

----- Round 2, with runtime_filter_mode=off -----
q1	6606	6579	6556	6556
q2	339	237	238	237
q3	2903	2790	2945	2790
q4	2035	1733	1761	1733
q5	5782	5778	5745	5745
q6	212	132	129	129
q7	2173	1798	1801	1798
q8	3398	3597	3556	3556
q9	8969	8741	8915	8741
q10	3579	3527	3501	3501
q11	591	513	495	495
q12	775	619	602	602
q13	10989	3267	3132	3132
q14	309	268	285	268
q15	517	467	472	467
q16	492	442	461	442
q17	1855	1649	1636	1636
q18	8250	7767	7854	7767
q19	1714	1481	1551	1481
q20	2060	1823	1857	1823
q21	5162	4999	4956	4956
q22	1089	1064	1037	1037
Total cold run time: 69799 ms
Total hot run time: 58892 ms

@doris-robot
Copy link

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

query1	1308	897	955	897
query2	6318	1927	1888	1888
query3	10844	4359	4475	4359
query4	61642	29434	23754	23754
query5	5206	457	453	453
query6	411	182	176	176
query7	5465	317	312	312
query8	319	228	229	228
query9	8479	2558	2564	2558
query10	463	272	253	253
query11	17590	15331	16209	15331
query12	163	107	106	106
query13	1519	433	423	423
query14	9845	7311	6897	6897
query15	204	196	176	176
query16	7218	496	494	494
query17	1175	590	582	582
query18	1907	334	319	319
query19	219	164	161	161
query20	118	110	111	110
query21	206	107	101	101
query22	4718	4813	4686	4686
query23	34701	33852	33930	33852
query24	6191	2960	2867	2867
query25	552	415	436	415
query26	670	178	172	172
query27	1895	368	388	368
query28	3888	2153	2110	2110
query29	680	454	440	440
query30	246	155	167	155
query31	1004	815	823	815
query32	79	55	58	55
query33	436	292	294	292
query34	934	507	517	507
query35	849	735	727	727
query36	1098	947	939	939
query37	106	65	69	65
query38	4050	3994	3973	3973
query39	1515	1481	1528	1481
query40	200	100	100	100
query41	49	48	53	48
query42	116	111	100	100
query43	529	472	508	472
query44	1180	819	848	819
query45	192	172	174	172
query46	1165	718	740	718
query47	2069	1927	1970	1927
query48	473	385	395	385
query49	741	411	398	398
query50	860	424	425	424
query51	7415	7343	7268	7268
query52	103	90	97	90
query53	261	183	182	182
query54	568	463	461	461
query55	79	77	75	75
query56	278	254	247	247
query57	1354	1231	1223	1223
query58	238	210	212	210
query59	3215	3074	3127	3074
query60	290	269	276	269
query61	113	114	115	114
query62	768	684	660	660
query63	213	191	182	182
query64	1349	674	669	669
query65	3262	3157	3173	3157
query66	703	290	307	290
query67	16004	15659	15804	15659
query68	4264	581	583	581
query69	445	264	268	264
query70	1198	1076	1067	1067
query71	359	251	244	244
query72	6371	4149	3957	3957
query73	756	345	353	345
query74	10507	8912	9212	8912
query75	3361	2610	2624	2610
query76	2071	1072	953	953
query77	535	280	264	264
query78	10648	9792	9612	9612
query79	2107	587	596	587
query80	1321	431	411	411
query81	516	223	225	223
query82	1242	91	87	87
query83	288	141	141	141
query84	275	77	81	77
query85	1012	307	293	293
query86	395	298	282	282
query87	4391	4216	4198	4198
query88	3745	2369	2271	2271
query89	413	304	297	297
query90	1953	185	180	180
query91	180	149	152	149
query92	63	48	50	48
query93	2652	557	555	555
query94	776	292	255	255
query95	358	266	254	254
query96	622	276	281	276
query97	3311	3195	3129	3129
query98	215	205	202	202
query99	1631	1280	1324	1280
Total cold run time: 315999 ms
Total hot run time: 196894 ms

@doris-robot
Copy link

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

query1	0.04	0.03	0.02
query2	0.07	0.04	0.03
query3	0.24	0.06	0.07
query4	1.61	0.10	0.10
query5	0.51	0.51	0.50
query6	1.13	0.73	0.72
query7	0.03	0.02	0.02
query8	0.04	0.04	0.03
query9	0.57	0.50	0.50
query10	0.55	0.55	0.59
query11	0.14	0.10	0.11
query12	0.14	0.12	0.11
query13	0.60	0.60	0.60
query14	0.77	0.79	0.80
query15	0.84	0.82	0.82
query16	0.38	0.39	0.39
query17	0.97	1.07	1.03
query18	0.23	0.22	0.22
query19	1.98	1.85	1.86
query20	0.01	0.01	0.01
query21	15.39	0.58	0.58
query22	2.39	1.97	2.04
query23	17.01	0.94	0.76
query24	3.36	1.67	2.42
query25	0.17	0.14	0.14
query26	0.62	0.14	0.13
query27	0.05	0.04	0.05
query28	8.29	0.51	0.48
query29	12.60	3.19	3.23
query30	0.25	0.07	0.06
query31	2.85	0.38	0.38
query32	3.25	0.48	0.46
query33	3.00	3.04	3.04
query34	17.02	4.53	4.48
query35	4.52	4.54	4.48
query36	0.67	0.48	0.47
query37	0.10	0.06	0.07
query38	0.05	0.04	0.04
query39	0.03	0.03	0.02
query40	0.16	0.12	0.13
query41	0.09	0.03	0.02
query42	0.04	0.02	0.02
query43	0.04	0.03	0.02
Total cold run time: 102.8 s
Total hot run time: 31.08 s

@github-actions
Copy link
Contributor

github-actions bot commented Aug 6, 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 Aug 6, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 6, 2025

PR approved by anyone and no changes requested.

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

@dataroaring dataroaring merged commit 3c66eda into apache:branch-3.0 Aug 12, 2025
23 checks passed
@gavinchou gavinchou mentioned this pull request Sep 1, 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. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants