[jira] [Commented] (CASSANDRA-10396) Simplify row cache invalidation code

["Sylvain Lebresne (JIRA)" <ji...@apache.org>]

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

Sylvain Lebresne commented on CASSANDRA-10396:
----------------------------------------------

Alright, attaching patch for trunk as there is no reason not to do this. I did a slight change of naming compared to the original patch by having the method we keep being name {{invalidateCachedPartition}} as it's more consistent with the method of the same name it defers to.

|| [patch|https://github.com/pcmanus/cassandra/commits/10396] || [utests|http://cassci.datastax.com/job/pcmanus-10396-testall/1/] || [dtests|http://cassci.datastax.com/job/pcmanus-10396-dtest/1/] ||


> Simplify row cache invalidation code
> ------------------------------------
>
>                 Key: CASSANDRA-10396
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10396
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Jonathan Ellis
>            Assignee: Jonathan Ellis
>            Priority: Trivial
>             Fix For: 3.x
>
>
> CFS.maybeUpdateRowCache and CFS.invalidateRowCache are nearly identical, except the latter does some unnecessary extra work looking up CFID.



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