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

branch-3.0: [chore](sink) enable_parallel_result_sink default value is changed to false #43933 #44241

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

github-actions[bot]
Copy link
Contributor

Cherry-picked from #43933

…to false (#43933)

### What problem does this PR solve?

Problem Summary:

For most queries, result sink will not become a performance bottleneck,
but the parallel result sink will increase the pressure of RPC between
fe and be.
@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.

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 Nov 19, 2024
@doris-robot
Copy link

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17567	7396	7290	7290
q2	2069	161	156	156
q3	10694	1047	1102	1047
q4	10560	754	742	742
q5	7768	2749	2814	2749
q6	235	146	146	146
q7	980	622	610	610
q8	9337	1914	1979	1914
q9	6558	6372	6328	6328
q10	6993	2246	2338	2246
q11	458	265	257	257
q12	417	211	210	210
q13	17765	2949	2963	2949
q14	246	210	202	202
q15	556	511	519	511
q16	664	599	593	593
q17	1000	571	621	571
q18	7241	6657	6647	6647
q19	1386	1048	1004	1004
q20	2861	2666	2655	2655
q21	3965	3180	3149	3149
q22	1072	979	979	979
Total cold run time: 110392 ms
Total hot run time: 42955 ms

----- Round 2, with runtime_filter_mode=off -----
q1	7383	7281	7271	7271
q2	345	237	233	233
q3	3046	2909	2928	2909
q4	2061	1795	1794	1794
q5	5659	5719	5829	5719
q6	220	143	147	143
q7	2163	1801	1804	1801
q8	3296	3439	3478	3439
q9	8906	8902	8869	8869
q10	3535	3532	3514	3514
q11	588	482	501	482
q12	791	581	602	581
q13	16472	3155	3102	3102
q14	299	289	291	289
q15	573	527	523	523
q16	716	671	676	671
q17	1868	1651	1630	1630
q18	8252	7663	7407	7407
q19	1731	1553	1546	1546
q20	2058	1829	1825	1825
q21	5480	5251	5153	5153
q22	1127	981	1015	981
Total cold run time: 76569 ms
Total hot run time: 59882 ms

@xinyiZzz xinyiZzz merged commit aff8d7c into branch-3.0 Nov 19, 2024
22 of 24 checks passed
@github-actions github-actions bot deleted the auto-pick-43933-branch-3.0 branch November 19, 2024 06:42
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.

3 participants