Skip to content

[fix](iceberg) disable default iceberg manifest cache and update test configurations #60027

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
morningman merged 1 commit into from
Jan 19, 2026
Merged

[fix](iceberg) disable default iceberg manifest cache and update test configurations #60027

morningman merged 1 commit into from
Jan 19, 2026

Conversation

@suxiaogang223
Copy link
Contributor

@suxiaogang223 suxiaogang223 commented Jan 19, 2026 •
edited
Loading

What problem does this PR solve?

relate pr: #59056

Release note

disable default iceberg manifest cache and update test configurations

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 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?

@suxiaogang223
Copy link
Contributor Author

suxiaogang223 commented Jan 19, 2026

run buildall

@doris-robot
Copy link

doris-robot commented Jan 19, 2026

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

------ Round 1 ----------------------------------
q1	17592	4199	4086	4086
q2	2040	359	240	240
q3	10146	1301	721	721
q4	10219	842	296	296
q5	7483	2022	1904	1904
q6	181	168	140	140
q7	936	782	649	649
q8	9285	1354	1091	1091
q9	4892	4468	4552	4468
q10	6707	1802	1401	1401
q11	517	298	282	282
q12	693	764	616	616
q13	17794	3823	3148	3148
q14	311	297	288	288
q15	563	508	527	508
q16	695	693	670	670
q17	651	779	503	503
q18	6654	6579	6783	6579
q19	1248	1023	662	662
q20	438	394	258	258
q21	3248	2631	2493	2493
q22	1188	1117	1050	1050
Total cold run time: 103481 ms
Total hot run time: 32053 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4293	4477	4245	4245
q2	333	405	387	387
q3	2198	2779	2465	2465
q4	1502	1863	1386	1386
q5	4405	4249	4363	4249
q6	270	176	133	133
q7	1976	2011	1766	1766
q8	2554	2443	2560	2443
q9	7144	7092	7118	7092
q10	2606	2748	2418	2418
q11	551	503	451	451
q12	713	733	658	658
q13	3701	4040	3155	3155
q14	259	284	282	282
q15	527	486	489	486
q16	604	662	607	607
q17	1099	1325	1335	1325
q18	7550	7414	7357	7357
q19	796	790	776	776
q20	1889	2003	1847	1847
q21	4495	4235	4163	4163
q22	1046	1071	978	978
Total cold run time: 50511 ms
Total hot run time: 48669 ms

@doris-robot
Copy link

doris-robot commented Jan 19, 2026

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

query5	4414	614	493	493
query6	321	222	219	219
query7	4209	445	276	276
query8	323	238	228	228
query9	8700	2936	2900	2900
query10	497	369	341	341
query11	15090	15142	15021	15021
query12	175	113	114	113
query13	1248	491	393	393
query14	5883	3017	2775	2775
query14_1	2720	2683	2656	2656
query15	200	190	170	170
query16	987	486	458	458
query17	1088	665	556	556
query18	2438	428	339	339
query19	225	225	191	191
query20	117	119	115	115
query21	222	134	114	114
query22	3798	4183	3830	3830
query23	15916	15785	15399	15399
query23_1	15607	15448	15536	15448
query24	7180	1533	1165	1165
query24_1	1148	1163	1171	1163
query25	516	416	404	404
query26	1238	256	153	153
query27	2782	439	274	274
query28	4580	2202	2189	2189
query29	722	525	408	408
query30	307	242	203	203
query31	774	629	563	563
query32	79	73	74	73
query33	519	346	303	303
query34	905	871	522	522
query35	707	771	701	701
query36	884	905	842	842
query37	129	95	83	83
query38	2729	2763	2641	2641
query39	774	770	740	740
query39_1	728	735	697	697
query40	215	136	115	115
query41	71	63	63	63
query42	105	104	101	101
query43	470	437	403	403
query44	1315	756	747	747
query45	182	193	176	176
query46	826	936	568	568
query47	1442	1498	1388	1388
query48	317	320	242	242
query49	620	429	341	341
query50	607	266	199	199
query51	3780	3773	3784	3773
query52	99	108	93	93
query53	295	329	271	271
query54	280	266	250	250
query55	83	79	76	76
query56	295	308	314	308
query57	1026	994	894	894
query58	275	281	257	257
query59	2074	2202	1917	1917
query60	336	325	312	312
query61	147	149	154	149
query62	378	349	328	328
query63	293	268	266	266
query64	4839	1245	946	946
query65	3849	3714	3736	3714
query66	1467	413	313	313
query67	15508	15555	15497	15497
query68	2451	1105	767	767
query69	460	373	323	323
query70	1002	961	919	919
query71	317	304	291	291
query72	5368	3355	3410	3355
query73	598	716	315	315
query74	8744	8780	8592	8592
query75	2753	2798	2431	2431
query76	2283	1047	662	662
query77	346	427	304	304
query78	9737	9959	9156	9156
query79	1066	893	612	612
query80	1331	569	467	467
query81	559	259	231	231
query82	1018	143	108	108
query83	356	258	246	246
query84	252	108	97	97
query85	894	478	418	418
query86	417	321	288	288
query87	2874	2877	2743	2743
query88	3458	2614	2557	2557
query89	403	342	342	342
query90	1946	171	168	168
query91	164	158	146	146
query92	86	72	70	70
query93	999	913	536	536
query94	649	319	308	308
query95	603	346	382	346
query96	648	496	228	228
query97	2351	2351	2308	2308
query98	218	206	199	199
query99	600	570	502	502
Total cold run time: 246408 ms
Total hot run time: 174006 ms

@doris-robot
Copy link

doris-robot commented Jan 19, 2026

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

query1	0.05	0.05	0.04
query2	0.10	0.05	0.05
query3	0.25	0.08	0.08
query4	1.61	0.11	0.11
query5	0.27	0.27	0.26
query6	1.15	0.65	0.65
query7	0.03	0.02	0.02
query8	0.06	0.04	0.04
query9	0.55	0.49	0.50
query10	0.55	0.55	0.57
query11	0.15	0.10	0.10
query12	0.14	0.11	0.11
query13	0.59	0.59	0.59
query14	0.94	0.94	0.93
query15	0.79	0.78	0.78
query16	0.41	0.40	0.39
query17	1.08	0.95	1.03
query18	0.23	0.21	0.21
query19	1.89	1.79	1.87
query20	0.02	0.01	0.02
query21	15.44	0.28	0.14
query22	5.11	0.05	0.05
query23	15.88	0.28	0.10
query24	1.46	0.26	0.59
query25	0.08	0.11	0.05
query26	0.14	0.13	0.14
query27	0.06	0.05	0.06
query28	3.52	1.08	0.89
query29	12.54	3.90	3.11
query30	0.28	0.13	0.13
query31	2.81	0.64	0.39
query32	3.25	0.56	0.46
query33	2.98	3.11	3.01
query34	16.04	5.08	4.44
query35	4.47	4.46	4.47
query36	0.66	0.51	0.48
query37	0.11	0.07	0.06
query38	0.07	0.04	0.04
query39	0.04	0.03	0.03
query40	0.16	0.14	0.13
query41	0.09	0.04	0.03
query42	0.04	0.02	0.02
query43	0.05	0.04	0.03
Total cold run time: 96.14 s
Total hot run time: 26.64 s

@hello-stephen
Copy link
Contributor

hello-stephen commented Jan 19, 2026

FE UT Coverage Report

Increment line coverage 100% (0/0) 🎉
Increment coverage report
Complete coverage report

@hello-stephen
Copy link
Contributor

hello-stephen commented Jan 19, 2026

FE Regression Coverage Report

Increment line coverage 100% (0/0) 🎉
Increment coverage report
Complete coverage report

@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

github-actions bot commented Jan 19, 2026

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

@github-actions
Copy link
Contributor

github-actions bot commented Jan 19, 2026

PR approved by anyone and no changes requested.

@suxiaogang223 suxiaogang223 changed the title fix: disable default iceberg manifest cache and update test configurations [fix](iceberg) disable default iceberg manifest cache and update test configurations Jan 19, 2026
@morningman morningman merged commit 6771470 into apache:master Jan 19, 2026
30 of 32 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 19, 2026
@suxiaogang223
[fix](iceberg) disable default iceberg manifest cache and update test…
f65dd2d 
… configurations (#60027)

relate pr: #59056
@github-actions github-actions bot mentioned this pull request Jan 19, 2026
Merged
yiguolei pushed a commit that referenced this pull request Jan 20, 2026
@github-actions @suxiaogang223
branch-4.0: [fix](iceberg) disable default iceberg manifest cache and…
4f87dc4 
… update test configurations #60027 (#60039)

Cherry-picked from #60027

Co-authored-by: Socrates <suyiteng@selectdb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@morningman morningman morningman approved these changes

@zhangstar333 zhangstar333 zhangstar333 approved these changes

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by one committer. dev/4.0.3-merged reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@suxiaogang223 @Thearas @doris-robot @hello-stephen @morningman @zhangstar333 @yiguolei