[GitHub] [incubator-doris] dataroaring commented on pull request #9350: remove explicit memtracker from scannode

[GitBox <gi...@apache.org>]

dataroaring commented on PR #9350:
URL: https://github.com/apache/incubator-doris/pull/9350#issuecomment-1117263656

   > `_block_mem_tracker` also exists in other places. I intentionally kept the previous memory track logic and tracked the more detailed block memory. If we don't need track block memory, or it will bring performance or other issues, then I will consider removing both. @dataroaring @morningman
   
   I removed it because its counterpart in olap_scan_node.cpp 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-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org