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

[opt](scanner profile) More counter for scanner #40144 #41059

Open
wants to merge 1 commit into
base: branch-3.0
Choose a base branch
from

Conversation

zhiqiang-hhhh
Copy link
Contributor

cherry pick from #40144

@zhiqiang-hhhh
Copy link
Contributor Author

run buildall

@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

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@dataroaring dataroaring force-pushed the pick_40144_to_upstream_branch-3.0 branch from 8fd32c0 to 5896cbe Compare September 21, 2024 01:17
@dataroaring
Copy link
Contributor

run buildall

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@zhiqiang-hhhh zhiqiang-hhhh force-pushed the pick_40144_to_upstream_branch-3.0 branch from 5896cbe to acb4ee8 Compare September 23, 2024 02:35
@zhiqiang-hhhh
Copy link
Contributor Author

run buildall

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@zhiqiang-hhhh zhiqiang-hhhh force-pushed the pick_40144_to_upstream_branch-3.0 branch from acb4ee8 to 6684aca Compare September 24, 2024 06:53
@zhiqiang-hhhh
Copy link
Contributor Author

run buildall

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

New profile metrics to monitor the schedule process of scanner.
```
VScanner:
    ...
    -  PeakMemoryUsage:  16.31  MB
    -  PeakRunningScanner:  1
    ...
```
In general, the value of `PeakMemoryUsage` is increased when any of the
scan tasks gets a block, decreased when ScanOperator consumes block from
block_queue.
@zhiqiang-hhhh zhiqiang-hhhh force-pushed the pick_40144_to_upstream_branch-3.0 branch from 6684aca to b1fc78d Compare September 25, 2024 11:57
@zhiqiang-hhhh
Copy link
Contributor Author

run buildall

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

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