-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[Feature](exectuor)Add scanbytes/scanrows condition, add move action in be #29337
Conversation
run buildall |
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.
clang-tidy made some suggestions
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G', run with scripts in https://github.com/apache/doris/tree/master/tools/tpch-tools
|
TPC-DS test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G', run with scripts in https://github.com/apache/doris/tree/master/tools/tpcds-tools
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
TeamCity be ut coverage result: |
@@ -29,7 +30,35 @@ void WorkloadActionCancelQuery::exec(WorkloadQueryInfo* query_info) { | |||
} | |||
|
|||
void WorkloadActionMoveQuery::exec(WorkloadQueryInfo* query_info) { | |||
LOG(INFO) << "[workload_schedule]move query action run group=" << _wg_name; | |||
QueryContext* query_ctx_ptr = query_info->_query_ctx_share_ptr.get(); |
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.
如果再执行这个exec 期间,query 结束了,quer context 析构了,怎么版?
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.
queryctx是存成一个sharedptr到WorkloadQueryInfo里了,WorkloadQueryInfo的生命周期是本次调度策略跑完所有的查询才会结束。不过这么一说感觉也有问题,queryctx可能析构的会很晚。感觉可以在每个查询策略匹配完之后就析构WorkloadQueryInfo
Add scanbytes/scanrows condition, add move action in be