-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[opt](resource) Implement ResourceContext
to manage task resources
#47274
base: master
Are you sure you want to change the base?
Conversation
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
ba42938
to
3d24170
Compare
run buildall |
3d24170
to
daf382f
Compare
run buildall |
daf382f
to
414079f
Compare
run buildall |
TeamCity be ut coverage result: |
414079f
to
43513d3
Compare
run buildall |
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 32805 ms
|
TPC-DS: Total hot run time: 193906 ms
|
ClickBench: Total hot run time: 30.96 s
|
43513d3
to
02cd0be
Compare
run buildall |
TPC-H: Total hot run time: 32534 ms
|
TPC-DS: Total hot run time: 193613 ms
|
ClickBench: Total hot run time: 30.92 s
|
TeamCity be ut coverage result: |
02cd0be
to
72ad8eb
Compare
run buildall |
72ad8eb
to
0ab9736
Compare
run buildall |
TPC-H: Total hot run time: 32335 ms
|
TPC-DS: Total hot run time: 184449 ms
|
ClickBench: Total hot run time: 30.97 s
|
TeamCity be ut coverage result: |
run buildall |
TPC-H: Total hot run time: 32696 ms
|
TPC-DS: Total hot run time: 196632 ms
|
ClickBench: Total hot run time: 30.13 s
|
TeamCity be ut coverage result: |
815a434
to
dcfaeab
Compare
run buildall |
TPC-H: Total hot run time: 32347 ms
|
TeamCity be ut coverage result: |
TPC-DS: Total hot run time: 191605 ms
|
ClickBench: Total hot run time: 30.77 s
|
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
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
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
What problem does this PR solve?
All tasks in Doris, query/load/compaction/etc., must have a unique
ResourceContext
to manage the resource usage of this Task, includingCpuContext
,MemoryContext
,IOContext
,WorkloadGroupContext
andTaskController
.What can
ResourceContext
do:Save task resource stat, such as profile counter of CPU/Memory/IO.
Hold resource objects such as
WorkloadGroupPtr
,MemTrackerPtr
.Provide methods for operating task resources, such as
revokable_bytes
,cancel task
.Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)