Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Binary protocol (incomplete) #40

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

outoftime
Copy link
Contributor

Hey Kelley,

This isn't an actual pull request, but I just wanted to let you know that I spent some time this weekend working on supporting the new CQL binary protocol in cassandra-cql. What I've got here so far is very much incomplete, particularly in terms of integrating with the rest of the library (I started messing around with it in isolation, but I definitely think it makes sense as a component of cassandra-cql rather than a separate library, as cassandra-cql does a lot that is agnostic to the Thrift protocol).

So far I've been trying to take as light a touch as possible in terms of modifying the existing cassandra-cql code, but it's looking like it might make sense to restructure a bit to isolate the parts of the library that are Thrift-specific (and provide alternate implementations that are compatible with the binary protocol). I'd welcome any input you have in terms of what I've got here so far and your preferences for how to move forward. I think it would be great to structure the library so that it would be straightforward to drop in Hector or the Java binary client in JRuby as well.

Not to worry if you don't have time to look at this right now -- I'll continue working on it either way -- but I just wanted to keep you up to speed to avoid any duplication of effort if nothing else.

Happy new year!
Mat

@brainopia
Copy link

Woah, nice one :)

@kreynolds
Copy link
Owner

@outoftime What is the state of this?

@outoftime
Copy link
Contributor Author

Very much experimental/work in progress. I'd love to spend more time on it but realistically it's probably not going to happen in the near future. Perhaps someone can pick up where I left off at some point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants