-
Notifications
You must be signed in to change notification settings - Fork 2
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
Option fix #4
base: master
Are you sure you want to change the base?
Option fix #4
Conversation
I think you had a discussion with alex about maintainership, was he wanting to continue maintaining cassandra-proto? |
Yes, we should push upstream once we get everything working. This is a largish change to the protocol parser. |
Any opportunity to upstream small changes that are correct in isolation should be taken. But I totally get that it can be hard to know what is correct until we get things fully working. |
zero-initialize buffer before passing to `Read`
Changes for Cassandra processing.
Adds Other version type.
Returns an Error if the parsing fails
Refactored to make parsing easier to read.