[ 
https://issues.apache.org/jira/browse/CASSANDRA-8897?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14514700#comment-14514700
 ] 

Benedict commented on CASSANDRA-8897:
-------------------------------------

Hmm. RandomAccessReader the only instances of this? If so could we just grab 
our buffer in {{reBuffer()}}?

If not, we should probably remove the logging altogether, or downgrade to 
DEBUG. We behave more optimally without sharing, but we also cope with it 
gracefully, so we should have some periodic review to ensure we're doing our 
best to avoid sharing, but we shouldn't probably consider it an error. I was 
initially a little concerned about having multiple operations with different 
lifetimes sharing a chunk, but really the chunks are very small, so it's not a 
big deal if their free only occurs at the end of the slowest operation. Only a 
single cross-over chunk can be affected per such operation, so it's not a lot 
to worry about unless there are many such operations and their lifetimes are 
highly varied. 




> Remove FileCacheService, instead pooling the buffers
> ----------------------------------------------------
>
>                 Key: CASSANDRA-8897
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-8897
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Benedict
>            Assignee: Stefania
>             Fix For: 3.0
>
>
> After CASSANDRA-8893, a RAR will be a very lightweight object and will not 
> need caching, so we can eliminate this cache entirely. Instead we should have 
> a pool of buffers that are page-aligned.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to