Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-9314

Overload SecondaryIndex#indexes to accept the column definition

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Normal
    • Resolution: Fixed
    • Fix Version/s: 2.0.15, 2.1.6, 2.2.0 beta 1
    • Component/s: None
    • Labels:
      None

      Description

      In order to avoid rebuilding indexes when no columns are actually indexed (originally implemented in CASSANDRA-9196), CASSANDRA-9281 hinted at overriding SecondaryIndex#getColumnDefs to "filter out" indexes that shouldn't be rebuilt; unfortunately, I believe this would be quite wrong for safety reasons, as it could have side effects on other methods relying on getColumnDefs returning a stable set, and API consistency ones, as it would be confusing to have getColumnDefs behave differently based on runtime conditions, possibly ignoring addColumnDef calls.

      I believe SecondaryIndex#indexes is still the best candidate to act as a filter predicate, and I propose to overload it to accept a column definition for all those cases (i.e. index rebuilding) missing a full cell name.

        Attachments

        1. 2.0-CASSANDRA-9314.txt
          7 kB
          Sergio Bossa

          Issue Links

            Activity

              People

              • Assignee:
                sbtourist Sergio Bossa
                Reporter:
                sbtourist Sergio Bossa
                Authors:
                Sergio Bossa
                Reviewers:
                Sam Tunnicliffe
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: