[Refactor] Remove meaningless profiler operator mem peaks

stdpain commented 2 days ago

Why I'm doing:

remove profile nodes:

    OperatorPeakMemoryUsage
    OperatorAllocatedMemoryUsage
    OperatorDeallocatedMemoryUsage

What I'm doing:

Fixes #issue

What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[x] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)
- [ ] I have added documentation for my new feature or new function
[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch
- [x] 3.4
- [x] 3.3
- [x] 3.2
- [x] 3.1
- [ ] 3.0

github-actions[bot] commented 1 day ago

[Java-Extensions Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] commented 1 day ago

[FE Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] commented 1 day ago

[BE Incremental Coverage Report]

:white_check_mark: pass : 6 / 6 (100.00%)

file detail

	path	covered_line	new_line	coverage	not_covered_line_detail
:large_blue_circle:	be/src/exec/pipeline/operator.cpp	1	1	100.00%	[]
:large_blue_circle:	be/src/exec/pipeline/operator.h	1	1	100.00%	[]
:large_blue_circle:	be/src/runtime/current_thread.h	4	4	100.00%	[]

wangsimo0 commented 1 day ago

ignore backport check: 3.3.7

stdpain commented 1 day ago

https://github.com/Mergifyio backport branch-3.4 branch-3.3 branch-3.2 branch-3.1

mergify[bot] commented 1 day ago

backport branch-3.4 branch-3.3 branch-3.2 branch-3.1

❌ No backport have been created

* [#53068 [Refactor] Remove meaningless profiler operator mem peaks (backport #53045)](https://github.com/StarRocks/starrocks/pull/53068) has been created for branch `branch-3.4` * Backport to branch `` failed GitHub error: ```Branch not found``` * [#53069 [Refactor] Remove meaningless profiler operator mem peaks (backport #53045)](https://github.com/StarRocks/starrocks/pull/53069) has been created for branch `branch-3.3` * Backport to branch `` failed GitHub error: ```Branch not found``` * [#53070 [Refactor] Remove meaningless profiler operator mem peaks (backport #53045)](https://github.com/StarRocks/starrocks/pull/53070) has been created for branch `branch-3.2` but encountered conflicts * [#53071 [Refactor] Remove meaningless profiler operator mem peaks (backport #53045)](https://github.com/StarRocks/starrocks/pull/53071) has been created for branch `branch-3.1` but encountered conflicts

yingtingdong commented 9 hours ago

ignore backport check: 3.2.13

StarRocks / starrocks