Skip to content

JAVA-1429: Prevent heartbeats until connection is fully initialized#830

Merged
adutra merged 2 commits into3.0.xfrom
java1429
Mar 24, 2017
Merged

JAVA-1429: Prevent heartbeats until connection is fully initialized#830
adutra merged 2 commits into3.0.xfrom
java1429

Conversation

@adutra
Copy link
Copy Markdown
Contributor

@adutra adutra commented Mar 23, 2017

No description provided.

@adutra adutra added this to the 3.0.8 milestone Mar 23, 2017
@adutra adutra force-pushed the java1429 branch 2 times, most recently from 62edcc5 to 505bb0d Compare March 23, 2017 13:22

@Override
public Authenticator newAuthenticator(InetSocketAddress host, String authenticator) throws AuthenticationException {
simulateBusyServer();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is very smart! Was wondering how we're going to test this and this great!

Copy link
Copy Markdown
Contributor

@tolbertam tolbertam Mar 23, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does a good job reproducing the issue (without the fix) too:

 18888  [cluster1-nio-worker-0] DEBUG com.datastax.driver.core.Host.STATES - Defuncting Connection[/127.0.1.1:65051-1, inFlight=0, closed=false] because: [/127.0.1.1:65051] Unexpected heartbeat response: ERROR PROTOCOL_ERROR: Unexpected message OPTIONS, expecting SASL_RESPONSE

@olim7t olim7t changed the base branch from 3.0.x to 3.x March 23, 2017 18:20
@olim7t olim7t changed the base branch from 3.x to 3.0.x March 23, 2017 18:24
@adutra adutra merged commit 3ba47ce into 3.0.x Mar 24, 2017
@adutra adutra deleted the java1429 branch March 24, 2017 09:39
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.

4 participants