-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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-wip] (memory tracker) (step6, End) Fix some details #9301
Merged
morningman
merged 6 commits into
apache:master
from
xinyiZzz:switch_tls_tracker6_fix_datils
May 10, 2022
Merged
[feature-wip] (memory tracker) (step6, End) Fix some details #9301
morningman
merged 6 commits into
apache:master
from
xinyiZzz:switch_tls_tracker6_fix_datils
May 10, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
xinyiZzz
force-pushed
the
switch_tls_tracker6_fix_datils
branch
2 times, most recently
from
April 29, 2022 10:39
a04f145
to
f2a8708
Compare
morningman
reviewed
Apr 30, 2022
xinyiZzz
force-pushed
the
switch_tls_tracker6_fix_datils
branch
2 times, most recently
from
May 6, 2022 03:13
22d4988
to
80f58df
Compare
morningman
added
kind/improvement
area/memory-consumption
dev/backlog
waiting to be merged in future dev branch
labels
May 7, 2022
morningman
previously approved these changes
May 7, 2022
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
github-actions
bot
added
the
approved
Indicates a PR has been approved by one committer.
label
May 7, 2022
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
xinyiZzz
force-pushed
the
switch_tls_tracker6_fix_datils
branch
from
May 9, 2022 02:04
28e4be0
to
5b56690
Compare
github-actions
bot
removed
the
approved
Indicates a PR has been approved by one committer.
label
May 9, 2022
morningman
approved these changes
May 10, 2022
zhengshiJ
pushed a commit
to zhengshiJ/incubator-doris
that referenced
this pull request
May 13, 2022
…9301) 1. Fix LoadTask, ChunkAllocator, TabletMeta, Brpc, the accuracy of memory track. 2. Modified some MemTracker names, deleted some unnecessary trackers, and improved readability. 3. More powerful MemTracker debugging capabilities. 4. Avoid creating TabletColumn temporary objects and improve BE startup time by 8%. 5. Fix some other details.
starocean999
pushed a commit
to starocean999/incubator-doris
that referenced
this pull request
May 19, 2022
…9301) 1. Fix LoadTask, ChunkAllocator, TabletMeta, Brpc, the accuracy of memory track. 2. Modified some MemTracker names, deleted some unnecessary trackers, and improved readability. 3. More powerful MemTracker debugging capabilities. 4. Avoid creating TabletColumn temporary objects and improve BE startup time by 8%. 5. Fix some other details.
englefly
pushed a commit
to englefly/incubator-doris
that referenced
this pull request
May 23, 2022
…9301) 1. Fix LoadTask, ChunkAllocator, TabletMeta, Brpc, the accuracy of memory track. 2. Modified some MemTracker names, deleted some unnecessary trackers, and improved readability. 3. More powerful MemTracker debugging capabilities. 4. Avoid creating TabletColumn temporary objects and improve BE startup time by 8%. 5. Fix some other details.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/memory-consumption
area/vectorization
dev/backlog
waiting to be merged in future dev branch
kind/improvement
reviewed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
Issue Number: close #7196
Problem Summary:
Checklist(Required)
Further comments
For related tests of impact and accuracy, can refer to the previous pr: #9145
In future: