[jira] [Updated] (CASSANDRA-11450) Should not search for the index of a column if the table is not using secondaryIndex.

["Dikang Gu (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/CASSANDRA-11450?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dikang Gu updated CASSANDRA-11450:
----------------------------------
    Reviewer: Marcus Eriksson
      Status: Patch Available  (was: Open)

> Should not search for the index of a column if the table is not using secondaryIndex.
> -------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-11450
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-11450
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Compaction
>            Reporter: Dikang Gu
>            Assignee: Dikang Gu
>             Fix For: 2.2.x
>
>
> We are not using secondary index in our cluster, but when I profile the compaction, I find that ~5.5% of the compaction time is spent on this line of the code function LazilyCompactedRow.Reducer.reduce():
>   if (cell.isLive() && !container.getColumn(cell.name()).equals(cell))
> before this line there is check to skip the look up, which seems to be not working:
>   // skip the index-update checks if there is no indexing needed since they are a bit expensive
>   if (indexer == SecondaryIndexManager.nullUpdater)
>       return;
> My patch is to set the indexer to be nullUpdater if the table has no associated index.
> Let me know if it's the right fix.



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