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

[enhacement](Nerids) fix bugs in struct info #27890

Merged
merged 1 commit into from
Dec 2, 2023

Conversation

keanji-x
Copy link
Contributor

@keanji-x keanji-x commented Dec 1, 2023

Proposed changes

fix bugs in struct info

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@keanji-x
Copy link
Contributor Author

keanji-x commented Dec 1, 2023

run buildall

@doris-robot
Copy link

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 44.08 seconds
stream load tsv: 564 seconds loaded 74807831229 Bytes, about 126 MB/s
stream load json: 18 seconds loaded 2358488459 Bytes, about 124 MB/s
stream load orc: 65 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select: 29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size: 17163917992 Bytes

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

github-actions bot commented Dec 1, 2023

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

Copy link
Contributor

github-actions bot commented Dec 1, 2023

PR approved by anyone and no changes requested.

@jackwener jackwener merged commit 934f37e into apache:master Dec 2, 2023
@doris-robot
Copy link

TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
Tpch sf100 test result on commit 8e6dfddd16978db4d5b7692f64665ecaf4e68fe3, data reload: true

run tpch-sf100 query with default conf and session variables
q1	4855	4617	4603	4603
q2	379	122	119	119
q3	1525	1323	1314	1314
q4	1173	956	977	956
q5	3265	3277	3262	3262
q6	244	127	124	124
q7	978	503	500	500
q8	2201	2207	2197	2197
q9	7071	6949	6938	6938
q10	3298	3393	3385	3385
q11	339	207	208	207
q12	362	208	209	208
q13	4645	3908	3889	3889
q14	255	216	206	206
q15	580	532	529	529
q16	430	381	395	381
q17	1013	660	607	607
q18	7994	7767	7371	7371
q19	1582	1540	1522	1522
q20	516	317	297	297
q21	3396	2961	2910	2910
q22	373	290	294	290
Total cold run time: 46474 ms
Total hot run time: 41815 ms

run tpch-sf100 query with default conf and set session variable runtime_filter_mode=off
q1	4584	4600	4567	4567
q2	314	198	237	198
q3	3734	3711	3720	3711
q4	2519	2513	2518	2513
q5	6190	6174	6163	6163
q6	245	126	127	126
q7	2568	2014	1946	1946
q8	3725	3667	3636	3636
q9	9480	9413	9417	9413
q10	4057	4182	4162	4162
q11	619	491	508	491
q12	796	645	624	624
q13	4340	3633	3662	3633
q14	277	247	259	247
q15	590	528	525	525
q16	534	455	513	455
q17	2118	2031	2096	2031
q18	9490	8914	8749	8749
q19	1771	1791	1770	1770
q20	2307	2005	1974	1974
q21	7249	7077	6852	6852
q22	635	575	568	568
Total cold run time: 68142 ms
Total hot run time: 64354 ms

XuJianxu pushed a commit to XuJianxu/doris that referenced this pull request Dec 14, 2023
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.

5 participants