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

Fix RUBY-1898 Socket reads may raise Timeout::Error #1482

Merged
merged 11 commits into from Aug 9, 2019

Conversation

p-mongo
Copy link
Contributor

@p-mongo p-mongo commented Aug 8, 2019

@p-mongo p-mongo changed the title RUBY-1898 Socket reads may raise Timeout::Error Fix RUBY-1898 Socket reads may raise Timeout::Error Aug 8, 2019
@p-mongo p-mongo merged commit fb85447 into mongodb:master Aug 9, 2019
@p-mongo p-mongo deleted the 1898 branch August 9, 2019 18:58
p-mongo added a commit that referenced this pull request Aug 9, 2019
* RUBY-1898 Correctly map select timeouts to Mongo::Error::TimeoutError

* RUBY-1898 Handle IO::WaitWritable when reading

* RUBY-1898 Raise more informative exceptions

* RUBY-1898 Rewrite the code to avoid weird failures

* Separate contexts to avoid unintentional connection reuse

* RUBY-1898 Handle IO::WaitWritable when reading

* Close all clients before

* Remove default 5 legacy read retries in the test suite since it interferes with assertions

* Remove all read retries for the connection timeout test

* Remove unneeded line

* Remove obsolete IO::WaitWritable rescue
p-mongo added a commit that referenced this pull request Aug 13, 2019
* RUBY-1898 Correctly map select timeouts to Mongo::Error::TimeoutError

* RUBY-1898 Handle IO::WaitWritable when reading

* RUBY-1898 Raise more informative exceptions

* RUBY-1898 Rewrite the code to avoid weird failures

* Separate contexts to avoid unintentional connection reuse

* RUBY-1898 Handle IO::WaitWritable when reading

* Close all clients before

* Remove default 5 legacy read retries in the test suite since it interferes with assertions

* Remove all read retries for the connection timeout test

* Remove unneeded line

* Remove obsolete IO::WaitWritable rescue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants