[ 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)