[
https://issues.apache.org/jira/browse/CASSANDRA-7622?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16434732#comment-16434732
]
Dinesh Joshi commented on CASSANDRA-7622:
-----------------------------------------
Hey [~cnlwsu],
Here's my feedback on the code so far -
* {{Schema#getVirtualTable}} - {{containsKey()}} followed by a {{get()}} is
unnecessary. Using {{get}} will have the same effect.
* {{VirtualSchema}} - the initializers for {{key}} and {{clustering}} fields
are unnecessary as you're overwriting them in the constructor. It would be a
good idea to make the fields final.
* {{VirtualTable#classFromName}}, {{TableMetaData#virtualClass}} - shouldn't
the error read differently? VirtualTable strategy class instead of Compaction
strategy class? Also there is no {{AbstractVirtualColumnFamilyStore}}. Am I
missing something here?
* {{InMemoryVirtualTable$SimpleVirtualCommand}} - Use finals for fields
* {{InMemoryVirtualTable$ResultReadState}} - Use finals for fields
* {{InMemoryVirtualTable$ResultReadState}} - line 258 - isn't the if check
redundant?
Nits -
* {{InMemoryVirtualTable}} - get rid of unused import
{{org.apache.cassandra.db.marshal.AbstractType;}}
> Implement virtual tables
> ------------------------
>
> Key: CASSANDRA-7622
> URL: https://issues.apache.org/jira/browse/CASSANDRA-7622
> Project: Cassandra
> Issue Type: Improvement
> Components: CQL
> 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
> CASSANDRA-7370.
> A possible implementation would be in terms of CASSANDRA-7443, but I am not
> presupposing.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
