[GitHub] [hbase] virajjasani commented on pull request #3215: HBASE-25698 Fixing IllegalReferenceCountException when using TinyLfuBlockCache

2021-05-24 Thread GitBox


virajjasani commented on pull request #3215:
URL: https://github.com/apache/hbase/pull/3215#issuecomment-847217261


   Okk looks like refCount was introduced with ByteBuff only, checking 
HBASE-22005


-- 
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:
us...@infra.apache.org




[GitHub] [hbase] virajjasani commented on pull request #3215: HBASE-25698 Fixing IllegalReferenceCountException when using TinyLfuBlockCache

2021-05-24 Thread GitBox


virajjasani commented on pull request #3215:
URL: https://github.com/apache/hbase/pull/3215#issuecomment-847215393


   > Will open an issue to address once an idea on how to address perf (this is 
secondary to fixing refcounting exceptions).
   
   refCount is not used for just L1 cache right? refCount is used only if we 
use Offheap BucketCache? And if this is the case, perhaps we could achieve 
CHM#get for L1 onheap cache and CHM#computeIfPresent (as is) for L1+L2 cache? 
WDYT @anoopsjohn @saintstack ?


-- 
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:
us...@infra.apache.org