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

Overload SecondaryIndex#indexes to accept the column definition

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Normal
    • Resolution: Fixed
    • 2.0.15, 2.1.6, 2.2.0 beta 1
    • None
    • 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

              sbtourist Sergio Bossa
              sbtourist Sergio Bossa
              Sergio Bossa
              Sam Tunnicliffe
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: