You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Aleksey Yeschenko (JIRA)" <ji...@apache.org> on 2015/07/16 21:33:05 UTC

[jira] [Commented] (CASSANDRA-9816) ALTER TABLE ADD validation is overly strict for re-added collection columns

    [ https://issues.apache.org/jira/browse/CASSANDRA-9816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630217#comment-14630217 ] 

Aleksey Yeschenko commented on CASSANDRA-9816:
----------------------------------------------

Pushed the fix and the test to a [GitHub branch|https://github.com/iamaleksey/cassandra/commits/9816] (dependent on CASSANDRA-6717 commits pending review).

> ALTER TABLE ADD validation is overly strict for re-added collection columns
> ---------------------------------------------------------------------------
>
>                 Key: CASSANDRA-9816
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9816
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Aleksey Yeschenko
>            Assignee: Aleksey Yeschenko
>            Priority: Minor
>             Fix For: 3.x
>
>
> {{AlterTableStatement}} ensures that we cannot re-add a collection column of a type that is incompatible with a previously existing, dropped one.
> It doesn't make an exception, however, for {{frozen}} collections, yet those should be allowed.
> Code in question:
> {code}
> if (dropped != null && (dropped.type == null || (dropped.type instanceof CollectionType && !type.isCompatibleWith(dropped.type))))
>     throw new InvalidRequestException(String.format("Cannot add a collection with the name %s " +
>                 "because a collection with the same name and a different type%s has already been used in the past",
>                 columnName, dropped.type == null ? "" : " (" + dropped.type.asCQL3Type() + ")"));
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)