[CASSANDRA-15663] DESCRIBE KEYSPACE does not properly quote table names

• Type: Bug
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 3.11.11, 4.0
• Component/s: CQL/Syntax
• Labels: pull-request-available
• Bug Category: Code - Bug - Unclear Impact
• Severity: Normal
• Complexity: Normal
• Discovered By: User Report
• Platform: All
• Since Version: 3.11.2

Source Control Link

https://github.com/apache/cassandra/commit/2a8e01858f3aa18e72806acfed5f9cfa9d6c6d03

Test and Documentation Plan

Thanks Alan!

Here are the patches:

• PR to update python driver in Cassandra 3.11 - https://github.com/apache/cassandra/pull/654
• PR to fix test_describe_functions broken by driver upgrade - https://github.com/apache/cassandra-dtest/pull/81

CI run  - https://ci-cassandra.apache.org/blue/organizations/jenkins/Cassandra-devbranch/detail/Cassandra-devbranch/154/tests/

I checked the rest of the failing tests manually. Test failures either do not reproduce locally or fail as well on the base branch.

 

• Assignee: Alex Sorokoumov
• Reporter: Oskar Liljeblad

Authors

Alex Sorokoumov

Reviewers

Andres de la Peña, Ekaterina Dimitrova

• Created: 26/Mar/20 06:13
• Updated: 15/Jul/21 09:15
• Resolved: 13/Jul/21 12:41

Description

How to reproduce (3.11.6) - cqlsh:

CREATE KEYSPACE test1 WITH replication = \{'class': 'SimpleStrategy', 'replication_factor': '1'} AND durable_writes = true;
CREATE TABLE test1."default" (id text PRIMARY KEY, data text, etag text);
DESCRIBE KEYSPACE test1;

Output will be:

CREATE TABLE test1.default (
 id text PRIMARY KEY,
 data text,
 etag text
) WITH [..]

Output should be:

CREATE TABLE test1."default" (
 id text PRIMARY KEY,
 data text,
 etag text
) WITH [..]

 If you try to run CREATE TABLE test1.default [..] you will get an error

SyntaxException: line 1:19 no viable alternative at input 'default' (CREATE TABLE test1.[default]...)

Oskar Liljeblad

 

Issue Links

• is fixed by: CASSANDRA-14825 Expose table schema for drivers
• is related to: CASSANDRA-15948 Multiple failures in cqlsh_tests.test_cqlsh.TestCqlsh
• links to: https://github.com/apache/cassandra-dtest/pull/148
• links to: PR 3.11
• links to: PR 4.0.0