[CASSANDRA-19889] Indexing a frozen collection that is the clustering key and reversed is rejected

• Type: Bug
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 4.0.15, 4.1.8, 5.0.2, 5.1
• Component/s: CQL/Interpreter
• Bug Category: Correctness - API / Semantic Implementation
• Severity: Critical
• Complexity: Low Hanging Fruit
• Discovered By: Fuzz Test
• Platform: All
• Since Version: 4.0

Source Control Link

https://github.com/apache/cassandra/commit/06a05ccde520925113edc92b28a4bd3b2bed2a29

Test and Documentation Plan

added test

• Assignee: Sunil Ramchandra Pawar
• Reporter: David Capwell

Authors

Sunil Ramchandra Pawar

Reviewers

David Capwell, Stefan Miklosovic

• Created: 03/Sep/24 18:04
• Updated: 30/Sep/24 12:24
• Resolved: 23/Sep/24 15:30

Description

CREATE TABLE tbl (
  pk int,
  ck frozen<list<int>>,
  value int,
  PRIMARY KEY(pk, ck)
)
WITH CLUSTERING ORDER BY (ck DESC)

If you index “ck” it will fail

CREATE INDEX ON tbl(FULL(ck));

The failure error is

Caused by: org.apache.cassandra.exceptions.InvalidRequestException: full() indexes can only be created on frozen collections
	at org.apache.cassandra.cql3.statements.schema.AlterSchemaStatement.ire(AlterSchemaStatement.java:222)
	at org.apache.cassandra.cql3.statements.schema.CreateIndexStatement.validateIndexTarget(CreateIndexStatement.java:250)
	at org.apache.cassandra.cql3.statements.schema.CreateIndexStatement.lambda$apply$1(CreateIndexStatement.java:177)

The reason? We have a ReverseType column! We must “unwrap” the type before this check...

Basic test

schemaChange(format("CREATE TABLE %s.tbl (\n" +
                            "  pk int,\n" +
                            "  ck frozen<list<int>>,\n" +
                            "  value int,\n" +
                            "  PRIMARY KEY(pk, ck)\n" +
                            ")\n" +
                            "WITH CLUSTERING ORDER BY (ck DESC)", KEYSPACE));
schemaChange(format("CREATE INDEX ON %s.tbl(FULL(ck));", KEYSPACE));

Issue Links

• links to: GitHub Pull Request #3557