You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "heguanhui (via GitHub)" <gi...@apache.org> on 2026/04/06 08:13:00 UTC

[PR] [bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf [doris]

heguanhui opened a new pull request, #62134:
URL: https://github.com/apache/doris/pull/62134

   ### 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 <!-- At least one of them must be included. -->
       - [ ] 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 <!-- Add your reason?  -->
   
   - Behavior changed:
       - [ ] No.
       - [ ] Yes. <!-- Explain the behavior change -->
   
   - Does this need documentation?
       - [ ] No.
       - [ ] Yes. <!-- Add document PR link here. eg: https://github.com/apache/doris-website/pull/1214 -->
   
   ### Check List (For Reviewer who merge this PR)
   
   - [ ] Confirm the release note
   - [ ] Confirm test cases
   - [ ] Confirm document
   - [ ] Add branch pick label <!-- Add branch pick label that this PR should merge into -->
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf [doris]

Posted by "morrySnow (via GitHub)" <gi...@apache.org>.
morrySnow merged PR #62134:
URL: https://github.com/apache/doris/pull/62134


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf [doris]

Posted by "heguanhui (via GitHub)" <gi...@apache.org>.
heguanhui commented on PR #62134:
URL: https://github.com/apache/doris/pull/62134#issuecomment-4192352112

   run buildall


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf [doris]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #62134:
URL: https://github.com/apache/doris/pull/62134#issuecomment-4196026532

   PR approved by anyone and no changes requested.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf [doris]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #62134:
URL: https://github.com/apache/doris/pull/62134#issuecomment-4196026434

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


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf [doris]

Posted by "Thearas (via GitHub)" <gi...@apache.org>.
Thearas commented on PR #62134:
URL: https://github.com/apache/doris/pull/62134#issuecomment-4191073349

   
   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/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?
       


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #62134:
URL: https://github.com/apache/doris/pull/62134#issuecomment-4192470335

   
   <details>
   <summary>TPC-H: <b>Total hot run time: 29117 ms</b></summary>
   
   ```
   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
   ```
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #62134:
URL: https://github.com/apache/doris/pull/62134#issuecomment-4192520060

   
   <details>
   <summary>TPC-DS: <b>Total hot run time: 178649 ms</b></summary>
   
   ```
   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
   ```
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [bugfix] fix show partitions command for external catalog validations incompatible with partitions tvf [doris]

Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #62134:
URL: https://github.com/apache/doris/pull/62134#issuecomment-4192695188

   # FE UT Coverage Report
   Increment line coverage `  100% (0/0)` :tada:
   [Increment coverage report](http://coverage.selectdb-in.cc/coverage/62134_d928b982b78147399f8db46e4510195c03661c4b/fe_increment_report/index.html)
   [Complete coverage report](http://coverage.selectdb-in.cc/coverage/62134_d928b982b78147399f8db46e4510195c03661c4b/fe_report/index.html)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org