Skip to content

[bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf - #62134

Merged
morrySnow merged 1 commit into
apache:masterfrom
heguanhui:bugfix/fix-show-partitions-command-with-iceberg-catalog-issue
Apr 7, 2026
Merged

[bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf#62134
morrySnow merged 1 commit into
apache:masterfrom
heguanhui:bugfix/fix-show-partitions-command-with-iceberg-catalog-issue

Conversation

@heguanhui

@heguanhuiheguanhui commented Apr 6, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

What problem does this PR solve?

fix show partitions command for external catalog validations incompatible with partitions tvf

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

@Thearas

Thearas commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

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?

@heguanhui
heguanhuiforce-pushed the bugfix/fix-show-partitions-command-with-iceberg-catalog-issue branch from e8104cf to d928b98CompareApril 6, 2026 12:52
@heguanhui

heguanhui commented Apr 6, 2026

Copy link
Copy Markdown
ContributorAuthor

run buildall

@doris-robot

doris-robot commented Apr 6, 2026

Copy link
Copy Markdown
TPC-H: Total hot run time: 29117 ms
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit d928b982b78147399f8db46e4510195c03661c4b, data reload: false
------ Round 1 ----------------------------------
orders	Doris	NULL	NULL	0	0	0	NULL	0	NULL	NULL	2023-12-26 18:27:23	2023-12-26 18:42:55	NULL	utf-8	NULL	NULL ============================================
q1	17672	3694	3684	3684
q2	q3	10689	911	625	625
q4	4677	469	357	357
q5	7451	1344	1150	1150
q6	189	168	138	138
q7	912	941	764	764
q8	9303	1427	1299	1299
q9	5381	5316	5225	5225
q10	6248	2044	1787	1787
q11	476	281	274	274
q12	635	417	293	293
q13	18081	2778	2177	2177
q14	286	285	265	265
q15	q16	852	869	788	788
q17	1044	1024	908	908
q18	6384	5721	5534	5534
q19	1158	1154	1071	1071
q20	536	422	283	283
q21	4143	2672	2130	2130
q22	536	421	365	365
Total cold run time: 96653 ms
Total hot run time: 29117 ms
----- Round 2, with runtime_filter_mode=off -----
orders	Doris	NULL	NULL	150000000	42	6422171781	NULL	22778155	NULL	NULL	2023-12-26 18:27:23	2023-12-26 18:42:55	NULL	utf-8	NULL	NULL ============================================
q1	4683	4526	4744	4526
q2	q3	4685	4748	4144	4144
q4	2091	2181	1386	1386
q5	4936	4950	5201	4950
q6	202	168	130	130
q7	2002	1814	1653	1653
q8	3352	3072	3052	3052
q9	8283	8252	8443	8252
q10	4462	4427	4262	4262
q11	635	410	378	378
q12	665	730	482	482
q13	2691	3047	2350	2350
q14	312	313	278	278
q15	q16	783	773	716	716
q17	1350	1336	1280	1280
q18	8240	7074	7097	7074
q19	1144	1188	1126	1126
q20	2220	2207	1947	1947
q21	6214	5578	4836	4836
q22	561	524	433	433
Total cold run time: 59511 ms
Total hot run time: 53255 ms

@doris-robot

doris-robot commented Apr 6, 2026

Copy link
Copy Markdown
TPC-DS: Total hot run time: 178649 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 d928b982b78147399f8db46e4510195c03661c4b, data reload: false
query5	4342	654	520	520
query6	362	237	211	211
query7	4255	587	347	347
query8	348	247	228	228
query9	8753	3897	3866	3866
query10	451	350	308	308
query11	6768	5521	5132	5132
query12	183	133	127	127
query13	1305	633	405	405
query14	5660	5188	4778	4778
query14_1	4142	4100	4097	4097
query15	213	203	183	183
query16	988	492	448	448
query17	972	777	647	647
query18	2467	510	384	384
query19	228	213	170	170
query20	140	129	132	129
query21	225	145	120	120
query22	14040	14860	14539	14539
query23	17948	17206	16725	16725
query23_1	17029	16686	16778	16686
query24	7453	1708	1333	1333
query24_1	1332	1398	1353	1353
query25	587	506	437	437
query26	1254	306	173	173
query27	2724	638	367	367
query28	4521	1880	1872	1872
query29	950	680	546	546
query30	296	235	195	195
query31	1090	1026	926	926
query32	97	72	69	69
query33	543	379	311	311
query34	1212	1140	683	683
query35	723	765	656	656
query36	1238	1261	1103	1103
query37	155	103	84	84
query38	3074	3030	2994	2994
query39	935	892	855	855
query39_1	837	846	834	834
query40	238	148	129	129
query41	62	57	57	57
query42	107	106	104	104
query43	311	313	279	279
query44 query45	206	193	183	183
query46	1142	1286	756	756
query47	2344	2311	2223	2223
query48	395	430	311	311
query49	640	536	419	419
query50	759	282	217	217
query51	4327	4294	4295	4294
query52	104	106	98	98
query53	250	262	206	206
query54	313	283	254	254
query55	94	92	89	89
query56	291	305	311	305
query57	1701	1822	1678	1678
query58	299	274	271	271
query59	2892	2990	2710	2710
query60	329	337	323	323
query61	155	152	155	152
query62	699	621	586	586
query63	241	206	193	193
query64	5351	1317	973	973
query65 query66	1544	469	367	367
query67	24370	24355	24312	24312
query68 query69	455	338	321	321
query70	1006	955	1005	955
query71	323	284	267	267
query72	2973	2724	2449	2449
query73	809	752	453	453
query74	9834	9723	9537	9537
query75	2727	2620	2333	2333
query76	2354	1135	776	776
query77	394	428	356	356
query78	11314	11300	10745	10745
query79	1344	1094	812	812
query80	674	613	530	530
query81	447	286	237	237
query82	241	162	124	124
query83	305	301	265	265
query84	299	154	126	126
query85	923	593	527	527
query86	352	320	332	320
query87	3256	3207	3110	3110
query88	3639	2715	2723	2715
query89	421	387	347	347
query90	2155	181	182	181
query91	178	168	138	138
query92	77	77	69	69
query93	964	940	552	552
query94	522	336	298	298
query95	646	370	424	370
query96	1017	795	325	325
query97	2681	2681	2526	2526
query98	236	237	226	226
query99	1068	1064	942	942
Total cold run time: 256497 ms
Total hot run time: 178649 ms

@hello-stephen

hello-stephen commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

FE UT Coverage Report

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

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Apr 7, 2026
@github-actions

github-actionsBot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actionsBot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@morrySnow
morrySnow merged commit 1668203 into apache:masterApr 7, 2026
29 of 31 checks passed
github-actionsBot pushed a commit that referenced this pull request Apr 7, 2026
@heguanhuiYour Name
[bugfix] fix show partitions command for external catalog validations…
46daa22
… incompatible with partitions tvf (#62134)
github-actionsBot pushed a commit that referenced this pull request Apr 7, 2026
@heguanhuiYour Name
[bugfix] fix show partitions command for external catalog validations…
8b9cc49
… incompatible with partitions tvf (#62134)
This was referenced Apr 7, 2026
Merged
Merged
yiguolei pushed a commit that referenced this pull request Apr 7, 2026
@github-actions@heguanhui
branch-4.1: [bugfix] fix show partitions command for external catalog…
6df2128
… validations incompatible with partitions tvf #62134 (#62152)
Cherry-picked from #62134
Co-authored-by: heguanhui <hgh_wy163mail@163.com>
iaorekhov-1980 pushed a commit to iaorekhov-1980/doris that referenced this pull request Apr 7, 2026
@heguanhui@iaorekhov-1980
[bugfix] fix show partitions command for external catalog validations…
cb1d7b4
… incompatible with partitions tvf (apache#62134)
yiguolei pushed a commit that referenced this pull request Apr 15, 2026
@github-actions@heguanhui
branch-4.0: [bugfix] fix show partitions command for external catalog…
65a3e30
… validations incompatible with partitions tvf #62134 (#62151)
Cherry-picked from #62134
Co-authored-by: heguanhui <hgh_wy163mail@163.com>
@mymeiyimymeiyi mentioned this pull request Apr 21, 2026
Merged
@morningmanmorningman mentioned this pull request Jun 3, 2026
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@starocean999starocean999starocean999 approved these changes
@morrySnowmorrySnowmorrySnow approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.dev/4.0.6-mergeddev/4.1.0-mergedreviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@heguanhui@Thearas@doris-robot@hello-stephen@starocean999@morrySnow@yiguolei