xinyiZzz commented on a change in pull request #8322:
URL: https://github.com/apache/incubator-doris/pull/8322#discussion_r823686866
##########
File path: be/src/exec/olap_scan_node.h
##########
@@ -244,6 +244,7 @@ class OlapScanNode : public ScanNode {
TResourceInfo* _resource_info;
int64_t _buffered_bytes;
+ std::shared_ptr<MemTracker> _scanner_mem_tracker;
Review comment:
_scanner_mem_tracker replaces the previous
runtime_state->fragment_mem_tracker(), both of which are also used to count the
memory of OlapScanner.
The difference is that,
1. The parent of runtime_state->fragment_mem_tracker() is
process_mem_tracker, which is used in segment_reader of segmentV1 to determine
whether the memory of OlapScanner has reached the upper limit.
2. The parent of _scanner_mem_tracker is query_mem_tracker, which is only
used for observation, and the judgment of the upper memory limit in
segment_reader is removed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
