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

Jonathan Ellis commented on CASSANDRA-6726:
-------------------------------------------

bq. I'd be inclined to include this patch in 2.1.1, otherwise we're unlikely to 
see much benefit from it, as I fully expect the new file format to not use 
these classes at all

It would have to be a pretty large gain to justify potentially destabilizing 
2.1 after all the effort we've put into QA for 2.1.0.

So I vote we either table this now and not sink more time into it, or if you 
think it's going to be a 20% win or more then we can benchmark it first.

> Recycle CompressedRandomAccessReader/RandomAccessReader buffers independently 
> of their owners, and move them off-heap when possible
> -----------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-6726
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-6726
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Benedict
>            Assignee: Branimir Lambov
>            Priority: Minor
>              Labels: performance
>             Fix For: 3.0
>
>         Attachments: cassandra-6726.patch
>
>
> Whilst CRAR and RAR are pooled, we could and probably should pool the buffers 
> independently, so that they are not tied to a specific sstable. It may be 
> possible to move the RAR buffer off-heap, and the CRAR sometimes (e.g. Snappy 
> may possibly support off-heap buffers)



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to