Nate McCall updated CASSANDRA-7622:
    Comment: was deleted

(was: Github user zznate commented on a diff in the pull request:

    --- Diff: 
src/java/org/apache/cassandra/cql3/statements/CreateTableStatement.java ---
    @@ -195,11 +217,27 @@ public TableMetadata toTableMetadata()
             private final Multiset<ColumnIdentifier> definedNames = 
             private final boolean ifNotExists;
    +        private final boolean isVirtual;
    +        private final String klass;
             public RawStatement(CFName name, boolean ifNotExists)
                 this.ifNotExists = ifNotExists;
    +            this.isVirtual = false;
    +            this.klass = null;
    +            assert isVirtual == (klass != null);
    --- End diff --
    Sorta OT, but we should just stop doing this and use `Preconditions` to 
throw IAEs. I hate the fact that a user can disable so much of our error 
checking w. a command line arg. 

> Implement virtual tables
> ------------------------
>                 Key: CASSANDRA-7622
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-7622
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Tupshin Harper
>            Assignee: Chris Lohfink
>            Priority: Major
>             Fix For: 4.x
> There are a variety of reasons to want virtual tables, which would be any 
> table that would be backed by an API, rather than data explicitly managed and 
> stored as sstables.
> One possible use case would be to expose JMX data through CQL as a 
> resurrection of CASSANDRA-3527.
> Another is a more general framework to implement the ability to expose yaml 
> configuration information. So it would be an alternate approach to 
> A possible implementation would be in terms of CASSANDRA-7443, but I am not 
> presupposing.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to