Skip to content
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.

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

[chore](cloud) Sync cloud modification #34213

Merged
merged 3 commits into from
Apr 30, 2024

Conversation

gavinchou
Copy link
Contributor

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Copy link
Contributor

PR approved by anyone and no changes requested.

@gavinchou
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 35.52% (8926/25130)
Line Coverage: 27.14% (73457/270638)
Region Coverage: 26.33% (37947/144112)
Branch Coverage: 23.13% (19333/83596)
Coverage Report: http://coverage.selectdb-in.cc/coverage/4adb5599ceeb5c8964a6769909e1cc1afa188c43_4adb5599ceeb5c8964a6769909e1cc1afa188c43/report/index.html

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17601	4427	4221	4221
q2	2013	194	188	188
q3	10607	1169	1182	1169
q4	10836	871	891	871
q5	7665	2729	2803	2729
q6	216	131	131	131
q7	1019	615	596	596
q8	9289	2141	2102	2102
q9	9332	6738	6568	6568
q10	8771	3723	3776	3723
q11	470	244	242	242
q12	398	213	211	211
q13	17763	2934	2946	2934
q14	263	226	227	226
q15	508	482	470	470
q16	509	378	383	378
q17	945	652	717	652
q18	8033	7406	7395	7395
q19	3247	1503	1498	1498
q20	644	301	305	301
q21	5004	3283	3341	3283
q22	325	272	282	272
Total cold run time: 115458 ms
Total hot run time: 40160 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4360	4249	4213	4213
q2	378	268	267	267
q3	2994	2753	2737	2737
q4	1840	1571	1588	1571
q5	5273	5293	5267	5267
q6	207	121	121	121
q7	2215	1883	1903	1883
q8	3187	3328	3319	3319
q9	8593	8532	8549	8532
q10	3883	3717	3673	3673
q11	598	484	485	484
q12	796	579	635	579
q13	16411	2948	2931	2931
q14	293	280	277	277
q15	507	474	472	472
q16	474	411	422	411
q17	1729	1492	1478	1478
q18	7789	7412	7427	7412
q19	1607	1526	1488	1488
q20	1966	1759	1779	1759
q21	4977	4865	4733	4733
q22	576	490	503	490
Total cold run time: 70653 ms
Total hot run time: 54097 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 186031 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 4adb5599ceeb5c8964a6769909e1cc1afa188c43, data reload: false

query1	909	362	346	346
query2	6435	2501	2409	2409
query3	6657	204	202	202
query4	25719	21346	21147	21147
query5	4134	425	429	425
query6	265	188	178	178
query7	4585	289	288	288
query8	254	198	194	194
query9	8508	2335	2307	2307
query10	439	256	239	239
query11	14870	14118	14263	14118
query12	143	89	83	83
query13	1626	360	370	360
query14	10530	7631	7663	7631
query15	255	167	168	167
query16	8133	262	259	259
query17	1841	554	548	548
query18	2090	270	264	264
query19	207	158	164	158
query20	95	87	87	87
query21	197	132	129	129
query22	4911	4787	4750	4750
query23	33769	33095	33385	33095
query24	11493	2981	2901	2901
query25	696	397	389	389
query26	1769	163	158	158
query27	3043	330	332	330
query28	7474	2033	2008	2008
query29	1043	631	618	618
query30	294	156	151	151
query31	988	752	736	736
query32	86	54	56	54
query33	750	260	257	257
query34	1095	483	493	483
query35	816	679	692	679
query36	1054	928	903	903
query37	278	69	70	69
query38	3316	3007	3027	3007
query39	1633	1571	1555	1555
query40	272	129	130	129
query41	47	43	40	40
query42	104	97	101	97
query43	588	547	532	532
query44	1236	740	750	740
query45	286	262	258	258
query46	1088	732	788	732
query47	1909	1851	1843	1843
query48	367	292	298	292
query49	1174	401	401	401
query50	766	395	385	385
query51	6904	6825	6728	6728
query52	112	90	95	90
query53	355	279	287	279
query54	319	231	239	231
query55	78	73	73	73
query56	244	236	219	219
query57	1221	1121	1132	1121
query58	231	196	198	196
query59	3546	3206	3359	3206
query60	256	236	252	236
query61	96	91	89	89
query62	647	439	448	439
query63	309	277	282	277
query64	9499	7232	7140	7140
query65	3082	3047	3044	3044
query66	1383	350	331	331
query67	15384	14982	14791	14791
query68	6816	550	569	550
query69	546	314	298	298
query70	1181	1120	1051	1051
query71	468	273	272	272
query72	7979	2600	2423	2423
query73	730	327	331	327
query74	6542	6058	6092	6058
query75	3963	2649	2706	2649
query76	4263	1053	984	984
query77	622	276	266	266
query78	10977	10290	10078	10078
query79	8347	530	529	529
query80	1490	444	445	444
query81	525	226	221	221
query82	777	98	93	93
query83	206	170	167	167
query84	265	86	84	84
query85	1462	288	305	288
query86	456	295	290	290
query87	3270	3059	3080	3059
query88	5273	2330	2327	2327
query89	557	379	391	379
query90	2003	191	190	190
query91	129	96	100	96
query92	66	48	48	48
query93	7361	532	509	509
query94	1157	179	177	177
query95	397	310	306	306
query96	629	266	262	262
query97	3204	2910	2938	2910
query98	252	223	226	223
query99	1294	852	868	852
Total cold run time: 307110 ms
Total hot run time: 186031 ms

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

Copy link
Contributor

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

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Apr 30, 2024
@dataroaring dataroaring merged commit d5e122a into apache:master Apr 30, 2024
25 of 29 checks passed
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.

4 participants