You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Sam Tunnicliffe (JIRA)" <ji...@apache.org> on 2014/08/11 16:05:12 UTC

[jira] [Created] (CASSANDRA-7742) Mutated ColumnFamily is not available to per-row indexes during update

Sam Tunnicliffe created CASSANDRA-7742:
------------------------------------------

             Summary: Mutated ColumnFamily is not available to per-row indexes during update
                 Key: CASSANDRA-7742
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-7742
             Project: Cassandra
          Issue Type: Bug
            Reporter: Sam Tunnicliffe
            Assignee: Sam Tunnicliffe


PerRowSecondaryIndex implementations are updated when SecondaryIndexManager.Updater.updateRowLevelIndexes is called from AtomicBTreeColumns.addAllWithSizeDelta. The StandardUpdater implementation does this by passing the key and cf from the incoming Mutation to the PRSI.index() method. However, the StandardUpdater instance created in Keyspace.apply() does not wrap the Mutation's cf as it is obtained using the wrong factory method. This results in the cf passed to the PRSI always being null. 

This isn't caught by PerRowSecondaryIndexTest because the stub index implementation there only verifies that the latest version of the cf can be obtained by the index reading from the base cf. 

Attaching a trivial patch to obtain an Updater which does wrap the mutated cf and also fix the test so it verifies both of the actual arguments passed to PRSI.index



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