Skip to content

Conversation

@github-actions
Copy link
Contributor

Cherry-picked from #49442

…49442)

We must add makeSureInitialized() in the fetchRowCount() method to
ensure the initialization of catalog related resources.
@github-actions github-actions bot requested a review from dataroaring as a code owner March 25, 2025 10:25
@Thearas
Copy link
Contributor

Thearas commented Mar 25, 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?

@dataroaring dataroaring reopened this Mar 25, 2025
@Thearas
Copy link
Contributor

Thearas commented Mar 25, 2025

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17567	6801	6564	6564
q2	2052	166	161	161
q3	10761	1069	1173	1069
q4	10561	756	684	684
q5	7730	2861	2796	2796
q6	218	135	133	133
q7	977	628	597	597
q8	9355	1928	2018	1928
q9	6629	6454	6428	6428
q10	7042	2282	2299	2282
q11	468	258	262	258
q12	396	207	207	207
q13	17773	3003	3000	3000
q14	244	205	216	205
q15	522	467	469	467
q16	672	596	620	596
q17	977	544	611	544
q18	7290	6666	6726	6666
q19	1405	1057	1136	1057
q20	463	209	201	201
q21	4057	3225	3316	3225
q22	1112	1073	1047	1047
Total cold run time: 108271 ms
Total hot run time: 40115 ms

----- Round 2, with runtime_filter_mode=off -----
q1	6801	6537	6545	6537
q2	328	236	225	225
q3	2918	2743	2914	2743
q4	2042	1834	1854	1834
q5	5792	5730	5732	5730
q6	215	128	129	128
q7	2257	1839	1802	1802
q8	3391	3533	3540	3533
q9	8762	8878	8884	8878
q10	3578	3552	3487	3487
q11	596	491	503	491
q12	821	635	615	615
q13	10328	3284	3212	3212
q14	309	274	268	268
q15	523	468	470	468
q16	688	649	661	649
q17	1857	1617	1620	1617
q18	8135	7921	7561	7561
q19	1663	1539	1540	1539
q20	2143	1889	1902	1889
q21	5528	5309	5293	5293
q22	1129	1041	1074	1041
Total cold run time: 69804 ms
Total hot run time: 59540 ms

@doris-robot
Copy link

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

query1	1350	936	914	914
query2	6252	2108	2123	2108
query3	10990	4491	4563	4491
query4	66929	27280	23959	23959
query5	5016	447	437	437
query6	397	171	177	171
query7	5512	307	302	302
query8	308	227	230	227
query9	8306	2624	2606	2606
query10	442	273	279	273
query11	17142	15183	16353	15183
query12	163	100	107	100
query13	1440	436	425	425
query14	10531	7700	7262	7262
query15	211	177	193	177
query16	7029	478	478	478
query17	1271	586	597	586
query18	1835	320	308	308
query19	201	158	161	158
query20	124	115	108	108
query21	211	103	103	103
query22	4691	4651	4650	4650
query23	34365	34174	34143	34143
query24	6126	2952	2906	2906
query25	522	398	407	398
query26	657	170	168	168
query27	1815	353	350	350
query28	4632	2486	2463	2463
query29	707	459	427	427
query30	247	167	166	166
query31	1022	823	848	823
query32	65	60	60	60
query33	400	290	304	290
query34	924	502	513	502
query35	887	735	752	735
query36	1127	980	995	980
query37	115	65	63	63
query38	4137	4123	4040	4040
query39	1548	1464	1481	1464
query40	218	107	99	99
query41	50	48	49	48
query42	111	100	101	100
query43	570	510	499	499
query44	1196	839	830	830
query45	183	173	169	169
query46	1151	721	742	721
query47	2069	1924	1971	1924
query48	470	374	385	374
query49	735	405	395	395
query50	854	436	435	435
query51	7449	7323	7005	7005
query52	102	87	90	87
query53	255	177	178	177
query54	549	465	442	442
query55	75	75	75	75
query56	260	232	233	232
query57	1270	1097	1118	1097
query58	211	207	206	206
query59	3261	3176	2971	2971
query60	270	254	250	250
query61	105	158	103	103
query62	774	681	666	666
query63	219	192	199	192
query64	1390	675	639	639
query65	3318	3197	3186	3186
query66	706	299	296	296
query67	16308	15652	15816	15652
query68	4103	582	584	582
query69	412	268	273	268
query70	1176	1140	1114	1114
query71	353	258	249	249
query72	6328	4072	4029	4029
query73	769	350	358	350
query74	10100	8979	9119	8979
query75	3383	2690	2675	2675
query76	1772	1131	1111	1111
query77	470	268	271	268
query78	10563	9684	9499	9499
query79	1120	599	605	599
query80	806	413	417	413
query81	521	247	243	243
query82	334	94	87	87
query83	160	143	143	143
query84	286	83	82	82
query85	824	306	296	296
query86	351	310	299	299
query87	4487	4397	4287	4287
query88	3959	2379	2334	2334
query89	420	298	293	293
query90	1939	184	194	184
query91	180	151	147	147
query92	66	49	49	49
query93	1471	563	564	563
query94	743	308	293	293
query95	353	256	250	250
query96	615	282	282	282
query97	3369	3220	3185	3185
query98	222	205	192	192
query99	1543	1299	1278	1278
Total cold run time: 316974 ms
Total hot run time: 198497 ms

@doris-robot
Copy link

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

query1	0.03	0.03	0.03
query2	0.07	0.03	0.03
query3	0.23	0.06	0.06
query4	1.62	0.10	0.10
query5	0.52	0.52	0.50
query6	1.14	0.73	0.72
query7	0.02	0.02	0.01
query8	0.04	0.03	0.04
query9	0.56	0.51	0.52
query10	0.55	0.55	0.56
query11	0.13	0.11	0.11
query12	0.16	0.11	0.12
query13	0.61	0.61	0.60
query14	2.72	2.77	2.77
query15	0.90	0.82	0.83
query16	0.38	0.39	0.37
query17	1.01	1.02	1.03
query18	0.24	0.22	0.21
query19	1.88	1.84	2.03
query20	0.01	0.01	0.01
query21	15.36	0.60	0.61
query22	2.54	2.98	1.56
query23	17.02	0.89	0.84
query24	2.98	1.48	1.48
query25	0.20	0.22	0.04
query26	0.42	0.13	0.13
query27	0.04	0.04	0.06
query28	9.68	0.49	0.50
query29	12.61	3.28	3.22
query30	0.24	0.06	0.06
query31	2.86	0.39	0.40
query32	3.24	0.47	0.46
query33	2.92	3.02	3.05
query34	16.91	4.55	4.49
query35	4.56	4.54	4.59
query36	0.66	0.49	0.48
query37	0.09	0.07	0.06
query38	0.04	0.04	0.04
query39	0.03	0.02	0.02
query40	0.17	0.13	0.13
query41	0.09	0.03	0.03
query42	0.04	0.02	0.02
query43	0.04	0.03	0.03
Total cold run time: 105.56 s
Total hot run time: 32.57 s

Copy link
Contributor

@dataroaring dataroaring left a comment

Choose a reason for hiding this comment

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

LGTM

@dataroaring dataroaring merged commit c780f2e into branch-3.0 Mar 27, 2025
23 of 24 checks passed
@github-actions github-actions bot deleted the auto-pick-49442-branch-3.0 branch March 27, 2025 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants