-
Notifications
You must be signed in to change notification settings - Fork 3.7k
add param: doris_exchange_instances to set parallel after exchange #1788
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
Conversation
Co-Authored-By: ZHAO Chun <buaa.zhaoc@gmail.com>
Co-Authored-By: ZHAO Chun <buaa.zhaoc@gmail.com>
Co-Authored-By: ZHAO Chun <buaa.zhaoc@gmail.com>
Co-Authored-By: ZHAO Chun <buaa.zhaoc@gmail.com>
|
@xionglei0 Hi, Thanks for your work! 1 Do you have the benchmark for this PR? 2 Could you use parallelExecInstanceNum instead of doris_exchange_instances? |
|
"parallel_fragment_exec_instance_num" means the parallel during scan_node |
|
doris_exchange_instances value need to be set according to your data size and UDAF, we usually configure it when table is duplication model |
Co-Authored-By: ZHAO Chun <buaa.zhaoc@gmail.com>
Co-Authored-By: ZHAO Chun <buaa.zhaoc@gmail.com>
Co-Authored-By: ZHAO Chun <buaa.zhaoc@gmail.com>
…i0/incubator-doris into doris_exchange_instances
Co-Authored-By: ZHAO Chun <buaa.zhaoc@gmail.com>
imay
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
… conf (apache#1788) The lead may lead to SEGV due to unreclaimable Txn, TxnKv, and Network.

doris_exchange_instances speed up the query especially useful when: