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

Using closed connection in the protobuf client #959

Merged
merged 1 commit into from
Nov 14, 2014
Merged

Using closed connection in the protobuf client #959

merged 1 commit into from
Nov 14, 2014

Conversation

dgnorton
Copy link
Contributor

If the remote server closed the connection, the protobuf client will constantly print this error message in the log when it should just drop the connection and wait for the remote server to reconnect:

[EROR] (github.com/influxdb/influxdb/coordinator.(*ProtobufClient).readResponses:174) Error while reading messsage size: &{%!d(string=use of closed network connection)}

@jvshahid jvshahid added this to the Next release milestone Sep 22, 2014
@jvshahid jvshahid modified the milestone: Next release Oct 9, 2014
@dgnorton
Copy link
Contributor

In case anyone else has the same thought I had at first, that it should check for errorClosing, here is a discussion on the topic https://code.google.com/p/go/issues/detail?id=4373.

toddboom added a commit that referenced this pull request Nov 14, 2014
Using closed connection in the protobuf client
@toddboom toddboom merged commit 3fb1d5d into master Nov 14, 2014
@toddboom toddboom removed the review label Nov 14, 2014
@toddboom toddboom deleted the fix-959 branch November 14, 2014 22:15
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.

None yet

3 participants