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

Auto-discover Cassandra protocol version #596

Merged

Conversation

3 participants
@ebyhr
Copy link
Contributor

commented Apr 6, 2019

Related to #278

@findepi Could you give me some advice to improve test coverage about protocol version?
ebyhr@dedc98d#commitcomment-33046480

@cla-bot cla-bot bot added the cla-signed label Apr 6, 2019

@ebyhr ebyhr force-pushed the ebyhr:cassandra/auto-discover-protocol-version branch from 11c0a2e to 11d796a Apr 7, 2019

@ebyhr ebyhr force-pushed the ebyhr:cassandra/auto-discover-protocol-version branch from 11d796a to 8941929 Apr 7, 2019

@electrum electrum requested a review from findepi Apr 25, 2019

@electrum electrum added this to the 311 milestone May 6, 2019

@electrum electrum merged commit ca78229 into prestosql:master May 6, 2019

2 checks passed

Travis CI - Pull Request Build Passed
Details
verification/cla-signed
Details
@findepi

This comment has been minimized.

Copy link
Member

commented May 6, 2019

thank you, @ebyhr @electrum !

@electrum electrum referenced this pull request May 14, 2019

Closed

Release notes for 311 #716

5 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.