[
https://issues.apache.org/jira/browse/CASSANDRA-1956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13180375#comment-13180375
]
Sylvain Lebresne commented on CASSANDRA-1956:
---------------------------------------------
bq. The only inefficiency as noted above are deletions which are hard to handle
without invalidation and now that we have them ttl columns.
Yes, though I'll note that a query cache have the exact same problem so it's
more a problem we have to deal with than anything else. And I'm good by
starting with just invalidating for deletion and TTL at first. On a second
iteration, I think we could improve the TTL case by keeping for each cached row
(at least those associated to a non-identity slice filter) a
firstColumnToExpireTimestamp. We would keep that to the smallest
localExpirationTime in the cached row. Gets and puts would just invalidate the
cache if now >= firstColumnToExpireTimestamp. But again, we don't have to
concern ourselves with that initially.
> Convert row cache to row+filter cache
> -------------------------------------
>
> Key: CASSANDRA-1956
> URL: https://issues.apache.org/jira/browse/CASSANDRA-1956
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Stu Hood
> Assignee: Vijay
> Priority: Minor
> Fix For: 1.2
>
> Attachments: 0001-1956-cache-updates-v0.patch,
> 0001-re-factor-row-cache.patch, 0001-row-cache-filter.patch,
> 0002-1956-updates-to-thrift-and-avro-v0.patch, 0002-add-query-cache.patch
>
>
> Changing the row cache to a row+filter cache would make it much more useful.
> We currently have to warn against using the row cache with wide rows, where
> the read pattern is typically a peek at the head, but this usecase would be
> perfect supported by a cache that stored only columns matching the filter.
> Possible implementations:
> * (copout) Cache a single filter per row, and leave the cache key as is
> * Cache a list of filters per row, leaving the cache key as is: this is
> likely to have some gotchas for weird usage patterns, and it requires the
> list overheard
> * Change the cache key to "rowkey+filterid": basically ideal, but you need a
> secondary index to lookup cache entries by rowkey so that you can keep them
> in sync with the memtable
> * others?
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira