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

Vijay commented on CASSANDRA-5357:
----------------------------------

Hi Jonathan, 

I pushed a basic version of Query cache to 
https://github.com/Vijay2win/cassandra/commits/query_cache .I am not sure if we 
still need RowCacheSentinel, but the attached removes it. Attached patch also 
has an option query_cache: true (if set to false, the whole row will always be 
cached). It will be nice to have fully off-heap Map/Cache (including the keys) 
but i am thinking to address it with a separate github project/patch (though 
IMHO, CHM may have contention in the segments for a big caches).

Let me know what you think about the patch, it might need some more cleanup.
                
> Query cache
> -----------
>
>                 Key: CASSANDRA-5357
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-5357
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Jonathan Ellis
>            Assignee: Vijay
>
> I think that most people expect the row cache to act like a query cache, 
> because that's a reasonable model.  Caching the entire partition is, in 
> retrospect, not really reasonable, so it's not surprising that it catches 
> people off guard, especially given the confusion we've inflicted on ourselves 
> as to what a "row" constitutes.
> I propose replacing it with a true query cache.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to