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

Jonathan Ellis edited comment on CASSANDRA-4795 at 1/15/13 5:04 AM:
--------------------------------------------------------------------

What if we required a static {{validate}} method on ACS subclasses that we call 
with the options and maybe CFMetaData?  Ugly, since there's no polymorphism on 
statics, but since we're doing ACS creation via reflection anyway it's not 
really a problem.

Edit: which is pretty much what you suggested with the difference being where 
the logic lives.  Either way WFM.
                
      was (Author: jbellis):
    What if we required a static {{validate}} method on ACS subclasses that we 
call with the options and maybe CFMetaData?  Ugly, since there's no 
polymorphism on statics, but since we're doing ACS creation via reflection 
anyway it's not really a problem.
                  
> replication, compaction, compression? options are not validated
> ---------------------------------------------------------------
>
>                 Key: CASSANDRA-4795
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-4795
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.1.0
>            Reporter: Brandon Williams
>            Assignee: Dave Brosius
>            Priority: Minor
>             Fix For: 1.2.1
>
>         Attachments: 4795.replication_strategy.txt
>
>
> When creating a keyspace and specifying strategy options, you can pass any 
> k/v pair you like.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to