dataroaring commented on PR #9350: URL: https://github.com/apache/incubator-doris/pull/9350#issuecomment-1118248438
> > > > _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: #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. Yep. I understand what you mean. IMHO, It should be case by case, e.g. for the OlapScanNode, the majority memory should be contributed by blocks, so value of block_mem_tracker is about equal value of ExecNode::_mem_tracker？ But for the AggregateNode, may be the majority memory should be contributed by hash table, so we should track memory usage of hash table. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
