Mailing-List: contact commits-help@cassandra.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cassandra.apache.org
Date: Thu, 3 Oct 2013 12:12:42 +0000 (UTC)
From: "Dave Brosius (JIRA)" <jira@apache.org>
To: commits@cassandra.apache.org
Message-ID: <JIRA.12672018.1380745936072.6416.1380802362798@arcas>
In-Reply-To: <JIRA.12672018.1380745936072@arcas>
References: <JIRA.12672018.1380745936072@arcas>
Subject: [jira] [Comment Edited] (CASSANDRA-6136) CQL should not allow an
 empty string as column identifier
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable


    [ https://issues.apache.org/jira/browse/CASSANDRA-6136?page=3Dcom.atlas=
sian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=3D=
13785028#comment-13785028 ]=20

Dave Brosius edited comment on CASSANDRA-6136 at 10/3/13 12:10 PM:
-------------------------------------------------------------------

version3 disallows at the parse layer.


was (Author: dbrosius):
version2 disallows at the parse layer.

> CQL should not allow an empty string as column identifier
> ---------------------------------------------------------
>
>                 Key: CASSANDRA-6136
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-6136
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Micha=C3=ABl Figui=C3=A8re
>            Assignee: Dave Brosius
>            Priority: Minor
>         Attachments: 6136.txt, 6136_v2.txt, 6136_v3.txt
>
>
> CQL currently allows users to create a table with an empty string as colu=
mn identifier:
> {code}
> CREATE TABLE t (k int primary key, "" int);
> {code}
> Which results in the following table:
> {code}
> CREATE TABLE t (
>   k int,
>   "" int,
>   PRIMARY KEY (k)
> ) WITH
>   bloom_filter_fp_chance=3D0.010000 AND
>   caching=3D'KEYS_ONLY' AND
>   comment=3D'' AND
>   dclocal_read_repair_chance=3D0.000000 AND
>   gc_grace_seconds=3D864000 AND
>   index_interval=3D128 AND
>   read_repair_chance=3D0.100000 AND
>   replicate_on_write=3D'true' AND
>   populate_io_cache_on_flush=3D'false' AND
>   default_time_to_live=3D0 AND
>   speculative_retry=3D'NONE' AND
>   memtable_flush_period_in_ms=3D0 AND
>   compaction=3D{'class': 'SizeTieredCompactionStrategy'} AND
>   compression=3D{'sstable_compression': 'SnappyCompressor'};
> {code}
> Empty strings are not allowed for keyspace and table identifiers though.
> I guess it's just a case that we haven't covered. Of course making it ill=
egal in a future version would be a breaking change, but nobody serious wou=
ld manually have chosen such an identifier...


--
This message was sent by Atlassian JIRA
(v6.1#6144)