xinyiZzz commented on PR #9350: URL: https://github.com/apache/incubator-doris/pull/9350#issuecomment-1118239190
> > > _block_mem_tracker > > > > > > So why remove `counterpart`, `counterpart` is just for `_block_mem_tracker`. > > It seems that _block_mem_tracker tracks memory for vectorized block? If so, we should track them via ExecNode::_mem_tracker， and it should be tracked via hook, right? btw, we should track mmap allocated memory. mmap has been tracked and introduced in pr: https://github.com/apache/incubator-doris/pull/9145 `_block_mem_tracker` only tracks memory for vectorized block, `ExecNode::_mem_tracker` includes other memory in ExecNode in addition to memory for vectorized block. This is the purpose of my previous design, so my reply above is whether it is necessary. -- 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
