mymeiyi commented on pull request #3318:
URL: https://github.com/apache/hbase/pull/3318#issuecomment-853523470


   > So the latest version change is moving the check for possible shipped() 
called out of the for loop. The cons here is that when we have wide rows with 
so many cells in it, this will delay the process of release of blocks and so 
release from cache. That is why it was kept in for loop.
   
   Yes, the block release may be delayed. There is a configuration named 
"hbase.hstore.compaction.kv.max" and default value is 10 to limit scan cell 
count in compaction. But if a cell size is huge, we can not avoid the delay.
   


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to