[
https://issues.apache.org/jira/browse/CASSANDRA-2478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13405325#comment-13405325
]
Yuki Morishita commented on CASSANDRA-2478:
-------------------------------------------
I'm +1 to put Sylvain's patch into 1.2. It works as described in spec, so
client developers can start developing on this new protocol.
But before commit, I suggest two changes to the patch:
* All of protocol related classes are placed under
org.apache.cassandra.cql3.transport, but I think it'd be better to keep them
under org.apache.cassandra.transport.
* Native transport service starts by default along with thrift service, but
isn't it better to turn off by default to indicate "for developing client only"?
And not necessary at this time, but it is nice to have unit test like CliTest,
based on debug client.
> Custom CQL protocol/transport
> -----------------------------
>
> Key: CASSANDRA-2478
> URL: https://issues.apache.org/jira/browse/CASSANDRA-2478
> Project: Cassandra
> Issue Type: New Feature
> Components: API, Core
> Reporter: Eric Evans
> Assignee: Sylvain Lebresne
> Priority: Minor
> Labels: cql
> Attachments: cql_binary_protocol, cql_binary_protocol-v2
>
>
> A custom wire protocol would give us the flexibility to optimize for our
> specific use-cases, and eliminate a troublesome dependency (I'm referring to
> Thrift, but none of the others would be significantly better). Additionally,
> RPC is bad fit here, and we'd do better to move in the direction of something
> that natively supports streaming.
> I don't think this is as daunting as it might seem initially. Utilizing an
> existing server framework like Netty, combined with some copy-and-paste of
> bits from other FLOSS projects would probably get us 80% of the way there.
--
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
