Skip to content

Conversation

@cambyzju
Copy link
Contributor

What problem does this PR solve?

this pr #32815 fixed in FE side, but not fixed it in BE side

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 Oct 22, 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?

@cambyzju
Copy link
Contributor Author

run buildall

@cambyzju
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

query1	1055	425	413	413
query2	6570	1736	1718	1718
query3	6771	236	221	221
query4	26102	23721	23171	23171
query5	4878	655	493	493
query6	346	246	239	239
query7	4644	490	311	311
query8	324	285	265	265
query9	8710	2628	2592	2592
query10	538	334	285	285
query11	15356	15082	14863	14863
query12	190	119	112	112
query13	1678	566	446	446
query14	11707	9181	9154	9154
query15	225	195	176	176
query16	7692	698	506	506
query17	1378	775	654	654
query18	2041	429	373	373
query19	248	238	201	201
query20	135	137	129	129
query21	225	139	157	139
query22	4464	4624	4601	4601
query23	34992	34265	33687	33687
query24	8697	2554	2532	2532
query25	602	580	513	513
query26	1291	302	172	172
query27	2762	555	358	358
query28	4629	2243	2188	2188
query29	842	683	509	509
query30	316	225	225	225
query31	985	828	806	806
query32	88	77	69	69
query33	598	401	373	373
query34	932	904	543	543
query35	847	820	755	755
query36	964	950	892	892
query37	130	103	80	80
query38	3524	3534	3454	3454
query39	1492	1417	1408	1408
query40	219	127	113	113
query41	60	62	57	57
query42	124	110	111	110
query43	496	494	451	451
query44	1227	737	748	737
query45	184	179	181	179
query46	895	989	634	634
query47	1741	1761	1687	1687
query48	393	426	323	323
query49	757	519	416	416
query50	653	681	417	417
query51	3935	3855	3856	3855
query52	110	111	101	101
query53	244	267	221	221
query54	598	587	532	532
query55	86	90	87	87
query56	323	299	313	299
query57	1164	1177	1090	1090
query58	293	279	292	279
query59	2627	2644	2562	2562
query60	353	349	328	328
query61	173	157	159	157
query62	816	713	648	648
query63	229	193	192	192
query64	4436	1139	918	918
query65	4001	3927	3949	3927
query66	1094	438	358	358
query67	15288	15308	14910	14910
query68	8983	890	633	633
query69	492	330	277	277
query70	1387	1332	1316	1316
query71	516	355	316	316
query72	5507	4873	4736	4736
query73	722	568	358	358
query74	9074	9148	9087	9087
query75	4064	3359	2850	2850
query76	3733	1160	789	789
query77	820	444	309	309
query78	9580	9805	8933	8933
query79	2556	808	610	610
query80	706	548	517	517
query81	486	262	226	226
query82	473	158	139	139
query83	299	267	243	243
query84	316	110	91	91
query85	895	449	419	419
query86	337	301	284	284
query87	3715	3759	3651	3651
query88	2846	2260	2228	2228
query89	419	331	317	317
query90	2036	219	233	219
query91	167	212	137	137
query92	86	63	64	63
query93	1968	968	649	649
query94	701	449	299	299
query95	411	318	312	312
query96	500	576	283	283
query97	2952	2946	2850	2850
query98	238	215	214	214
query99	1515	1388	1295	1295
Total cold run time: 279987 ms
Total hot run time: 189864 ms

@doris-robot
Copy link

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

query1	0.06	0.06	0.05
query2	0.09	0.05	0.06
query3	0.26	0.08	0.08
query4	1.61	0.12	0.12
query5	0.27	0.27	0.26
query6	1.19	0.66	0.65
query7	0.04	0.03	0.03
query8	0.05	0.04	0.04
query9	0.63	0.54	0.52
query10	0.58	0.58	0.56
query11	0.16	0.11	0.12
query12	0.15	0.13	0.12
query13	0.63	0.60	0.60
query14	1.01	1.02	1.00
query15	0.84	0.83	0.86
query16	0.41	0.38	0.39
query17	1.00	1.04	1.02
query18	0.22	0.20	0.19
query19	1.88	1.82	1.81
query20	0.02	0.01	0.01
query21	15.43	0.19	0.14
query22	5.05	0.06	0.04
query23	15.71	0.26	0.10
query24	2.82	1.21	0.79
query25	0.09	0.07	0.07
query26	0.14	0.13	0.14
query27	0.06	0.06	0.05
query28	5.73	1.15	0.93
query29	12.55	3.91	3.24
query30	0.30	0.13	0.12
query31	2.82	0.59	0.39
query32	3.26	0.55	0.48
query33	3.11	3.13	3.09
query34	15.84	5.09	4.52
query35	4.57	4.54	4.56
query36	0.68	0.51	0.50
query37	0.11	0.06	0.06
query38	0.06	0.04	0.04
query39	0.04	0.03	0.03
query40	0.18	0.15	0.14
query41	0.09	0.04	0.03
query42	0.04	0.03	0.03
query43	0.04	0.04	0.04
Total cold run time: 99.82 s
Total hot run time: 27.93 s

@cambyzju
Copy link
Contributor Author

run p0

Copy link
Contributor

@morningman morningman 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
Copy link
Contributor

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

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

PR approved by anyone and no changes requested.

Copy link
Member

@CalvinKirs CalvinKirs left a comment

Choose a reason for hiding this comment

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

Have you tried the new version, master/3.1 has fixed this issue.

@cambyzju
Copy link
Contributor Author

Have you tried the new version, master/3.1 has fixed this issue.

有相关的pr,或者关键字吗?

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.x dev/3.1.x dev/4.0.x reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants