You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Dave Brosius (Commented) (JIRA)" <ji...@apache.org> on 2012/04/12 06:54:56 UTC

[jira] [Commented] (CASSANDRA-4046) when creating keyspace with simple strategy, it should only acception "replication_factor" as an option

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

Dave Brosius commented on CASSANDRA-4046:
-----------------------------------------

Implementations of AbstractReplicationStrategy.validateOptions only does positive validations, negative validations for these would need to be implemented for this issue.
                
> when creating keyspace with simple strategy, it should only acception "replication_factor" as an option
> -------------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-4046
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-4046
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Jackson Chung
>            Priority: Minor
>
> currently I could do this:
> {panel}
> [default@unknown] create keyspace test
> ...     with placement_strategy = 'SimpleStrategy'
> ...     and strategy_options = \{DC : testdc, replication_factor :1\};
> ebc5f430-6d47-11e1-0000-edee3ea2cbff
> Waiting for schema agreement...
> ... schemas agree across the cluster
> [default@unknown] 
> {panel}
> while i don't think this creates any "problem" in terms of the actual replication being used for the CL , we probably should acknowledge to the user that "DC : testdc" is not an valid option for the SimpleStrategy.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira