Skip to content

[fix](compile) Fix compile failure#60008

Merged
Gabriel39 merged 1 commit intoapache:masterfrom
Gabriel39:fix_0119
Jan 19, 2026
Merged

[fix](compile) Fix compile failure#60008
Gabriel39 merged 1 commit intoapache:masterfrom
Gabriel39:fix_0119

Conversation

@Gabriel39
Copy link
Contributor

What problem does this PR solve?

Introduced by #59206

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

@Gabriel39 Gabriel39 requested a review from zclllyybb as a code owner January 19, 2026 02:23
@Thearas
Copy link
Contributor

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

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

1 similar comment
@hello-stephen
Copy link
Contributor

skip buildall

@Gabriel39 Gabriel39 merged commit 54b9d70 into apache:master Jan 19, 2026
29 of 31 checks passed
@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17602	4310	4063	4063
q2	2127	348	252	252
q3	10094	1297	701	701
q4	10213	866	306	306
q5	7479	2096	1871	1871
q6	195	170	139	139
q7	945	784	655	655
q8	9275	1436	1200	1200
q9	5003	4663	4787	4663
q10	6740	1852	1435	1435
q11	520	311	284	284
q12	731	749	605	605
q13	17779	3882	3060	3060
q14	295	290	277	277
q15	587	521	518	518
q16	666	700	626	626
q17	644	744	603	603
q18	6734	6389	7008	6389
q19	1456	1089	731	731
q20	420	387	249	249
q21	3339	2791	2479	2479
q22	1116	1117	1015	1015
Total cold run time: 103960 ms
Total hot run time: 32121 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4322	4323	4340	4323
q2	339	405	310	310
q3	2236	2810	2476	2476
q4	1522	1887	1528	1528
q5	4494	4502	4313	4313
q6	220	178	130	130
q7	1982	1940	1757	1757
q8	2555	2392	2396	2392
q9	7288	7040	7078	7040
q10	2431	2550	2101	2101
q11	580	455	436	436
q12	667	682	580	580
q13	3352	3816	3055	3055
q14	280	303	272	272
q15	522	495	475	475
q16	618	640	620	620
q17	1108	1368	1392	1368
q18	7305	7473	7248	7248
q19	910	808	831	808
q20	1869	1948	1785	1785
q21	4570	4242	4140	4140
q22	1082	1000	997	997
Total cold run time: 50252 ms
Total hot run time: 48154 ms

@doris-robot
Copy link

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

query5	4411	643	483	483
query6	324	228	217	217
query7	4217	486	272	272
query8	355	262	258	258
query9	8702	2931	2935	2931
query10	497	382	341	341
query11	15127	15244	15047	15047
query12	171	116	114	114
query13	1262	493	404	404
query14	5961	3040	2858	2858
query14_1	2674	2628	2661	2628
query15	202	187	168	168
query16	981	495	450	450
query17	1057	628	527	527
query18	2407	436	324	324
query19	217	230	184	184
query20	122	114	117	114
query21	210	135	110	110
query22	3887	4258	4075	4075
query23	15848	15684	15401	15401
query23_1	15525	15476	15393	15393
query24	7139	1552	1181	1181
query24_1	1167	1167	1201	1167
query25	522	416	406	406
query26	1243	265	148	148
query27	2779	443	283	283
query28	4555	2205	2189	2189
query29	750	519	421	421
query30	310	242	205	205
query31	756	609	562	562
query32	84	74	74	74
query33	542	344	310	310
query34	889	863	535	535
query35	714	741	686	686
query36	858	886	817	817
query37	141	99	81	81
query38	2713	2753	2614	2614
query39	758	756	720	720
query39_1	707	723	723	723
query40	231	132	118	118
query41	69	63	59	59
query42	104	99	100	99
query43	444	464	405	405
query44	1368	762	755	755
query45	187	188	177	177
query46	864	955	579	579
query47	1397	1479	1373	1373
query48	316	331	242	242
query49	611	416	363	363
query50	618	265	195	195
query51	3811	3720	3730	3720
query52	101	105	92	92
query53	293	327	264	264
query54	283	269	254	254
query55	80	83	77	77
query56	312	300	305	300
query57	1017	1011	942	942
query58	268	255	265	255
query59	2061	2186	2152	2152
query60	342	318	308	308
query61	150	149	138	138
query62	380	349	337	337
query63	293	268	259	259
query64	4934	1249	976	976
query65	3844	3749	3776	3749
query66	1479	438	339	339
query67	15534	15590	15388	15388
query68	2529	1114	764	764
query69	450	377	324	324
query70	1004	985	959	959
query71	328	320	297	297
query72	5456	3375	3393	3375
query73	612	733	314	314
query74	8733	8709	8561	8561
query75	2762	2831	2498	2498
query76	2288	1062	666	666
query77	363	406	331	331
query78	9795	10134	9192	9192
query79	1081	906	593	593
query80	896	592	525	525
query81	508	266	234	234
query82	1372	147	111	111
query83	343	252	239	239
query84	257	114	87	87
query85	1002	470	422	422
query86	369	303	290	290
query87	2869	2870	2779	2779
query88	3509	2604	2570	2570
query89	410	363	327	327
query90	1768	172	173	172
query91	166	164	139	139
query92	72	75	73	73
query93	956	906	550	550
query94	568	328	307	307
query95	588	334	392	334
query96	641	520	232	232
query97	2344	2386	2315	2315
query98	211	198	198	198
query99	587	558	504	504
Total cold run time: 245930 ms
Total hot run time: 174906 ms

@doris-robot
Copy link

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

query1	0.05	0.04	0.05
query2	0.10	0.04	0.04
query3	0.25	0.08	0.08
query4	1.60	0.11	0.11
query5	0.27	0.26	0.27
query6	1.15	0.67	0.65
query7	0.03	0.03	0.02
query8	0.06	0.04	0.04
query9	0.58	0.48	0.48
query10	0.54	0.53	0.55
query11	0.15	0.09	0.09
query12	0.14	0.10	0.11
query13	0.60	0.58	0.58
query14	0.96	0.95	0.94
query15	0.78	0.77	0.79
query16	0.43	0.38	0.40
query17	1.02	1.02	1.06
query18	0.23	0.21	0.21
query19	1.91	1.83	1.84
query20	0.02	0.01	0.02
query21	15.42	0.26	0.15
query22	5.12	0.05	0.05
query23	15.87	0.28	0.10
query24	1.94	0.67	0.30
query25	0.09	0.08	0.05
query26	0.15	0.14	0.13
query27	0.11	0.07	0.05
query28	5.00	1.06	0.88
query29	12.54	3.94	3.11
query30	0.27	0.13	0.12
query31	2.82	0.64	0.39
query32	3.24	0.57	0.46
query33	3.03	2.94	3.09
query34	16.14	5.06	4.48
query35	4.50	4.46	4.44
query36	0.65	0.50	0.49
query37	0.11	0.07	0.06
query38	0.07	0.05	0.04
query39	0.05	0.03	0.03
query40	0.17	0.14	0.14
query41	0.09	0.04	0.03
query42	0.04	0.02	0.02
query43	0.04	0.03	0.04
Total cold run time: 98.33 s
Total hot run time: 26.67 s

@doris-robot
Copy link

BE UT Coverage Report

Increment line coverage 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 52.43% (19051/36337)
Line Coverage 35.72% (176452/493958)
Region Coverage 32.29% (136715/423398)
Branch Coverage 33.16% (59058/178104)

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