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

[test](multi-catalog)fix fe ut, now create cluster be node is not required #28746

Closed
wants to merge 1 commit into from

Conversation

wsjz
Copy link
Contributor

@wsjz wsjz commented Dec 20, 2023

Proposed changes

fix pipeline fe ut problem, create cluster repeatedly

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...

Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Dec 21, 2023
Copy link
Contributor

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

Copy link
Contributor

PR approved by anyone and no changes requested.

@wsjz
Copy link
Contributor Author

wsjz commented Dec 21, 2023

run buildall

@doris-robot
Copy link

TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'

Tpch sf100 test result on commit 1b5fef179efb1373848611ca56f15a99ca08f238, data reload: false

run tpch-sf100 query with default conf and session variables
q1	4750	4427	4487	4427
q2	374	178	160	160
q3	1450	1281	1200	1200
q4	1114	904	899	899
q5	3162	3207	3171	3171
q6	253	129	130	129
q7	984	488	492	488
q8	2202	2204	2194	2194
q9	6711	6702	6655	6655
q10	3249	3317	3309	3309
q11	309	201	192	192
q12	360	219	213	213
q13	4571	3849	3838	3838
q14	248	223	220	220
q15	562	526	533	526
q16	456	400	399	399
q17	1019	581	549	549
q18	7227	7023	6937	6937
q19	1527	1352	1440	1352
q20	508	348	325	325
q21	3034	2669	2712	2669
q22	364	294	302	294
Total cold run time: 44434 ms
Total hot run time: 40146 ms

run tpch-sf100 query with default conf and set session variable runtime_filter_mode=off
q1	4400	4383	4395	4383
q2	272	171	180	171
q3	3547	3525	3528	3525
q4	2389	2377	2371	2371
q5	5745	5763	5756	5756
q6	247	122	122	122
q7	2382	1895	1888	1888
q8	3543	3531	3533	3531
q9	9040	9055	8986	8986
q10	3920	4023	4013	4013
q11	488	378	381	378
q12	768	610	611	610
q13	4296	3577	3585	3577
q14	285	255	253	253
q15	574	515	517	515
q16	524	466	474	466
q17	1904	1844	1860	1844
q18	8716	8317	8283	8283
q19	1740	1740	1758	1740
q20	2267	1962	1939	1939
q21	6565	6233	6205	6205
q22	526	464	436	436
Total cold run time: 64138 ms
Total hot run time: 60992 ms

@wsjz wsjz closed this Dec 21, 2023
@wsjz
Copy link
Contributor Author

wsjz commented Dec 21, 2023

#28722 have fixed

@doris-robot
Copy link

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 44.78 seconds
stream load tsv: 564 seconds loaded 74807831229 Bytes, about 126 MB/s
stream load json: 19 seconds loaded 2358488459 Bytes, about 118 MB/s
stream load orc: 66 seconds loaded 1101869774 Bytes, about 15 MB/s
stream load parquet: 32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select: 29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size: 17183605861 Bytes

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.

3 participants