[CDRIVER-594] Make all of the timeouts and intervals in SS/SDAM work correctly Created: 27/Mar/15 Updated: 08/Jan/24 Resolved: 07/Apr/15 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.2-beta0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Mira Carey | Assignee: | Mira Carey |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Description |
|
Nothing was really right (cond_timedwait was a bad implementation, a bunch of the timeouts were using microseconds and milliseconds interchangably, etc. etc) We didn't let you set any of the configurable parameters, etc, etc. Fix it all |
| Comments |
| Comment by Githook User [ 07/Oct/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@emptysquare.net'}Message: Merge pull request #218 from ajdavis/timeout-fix
| |||||||||||||||||||||||
| Comment by Githook User [ 07/Oct/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: Cancel after first blocking scan's timeout, not the second's. | |||||||||||||||||||||||
| Comment by Githook User [ 07/Oct/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@emptysquare.net'}Message: Merge pull request #213 from hanumantmk/
| |||||||||||||||||||||||
| Comment by Githook User [ 07/Oct/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@emptysquare.net'}Message: Merge pull request #213 from hanumantmk/
| |||||||||||||||||||||||
| Comment by Githook User [ 07/Oct/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'hanumantmk', u'name': u'Jason Carey (hanumantmk)', u'email': u'jcarey@argv.me'}Message: | |||||||||||||||||||||||
| Comment by Githook User [ 11/Apr/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@emptysquare.net'}Message: Merge pull request #218 from ajdavis/timeout-fix
| |||||||||||||||||||||||
| Comment by Githook User [ 11/Apr/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: Cancel after first blocking scan's timeout, not the second's. | |||||||||||||||||||||||
| Comment by Hannes Magnusson [ 08/Apr/15 ] | |||||||||||||||||||||||
|
mira.carey@mongodb.com before the latest commit to
after:
both cases, 99.9% of cpu the entire time – EDIT: Whops, debug.txt was appended to – updated the grep count | |||||||||||||||||||||||
| Comment by Mira Carey [ 08/Apr/15 ] | |||||||||||||||||||||||
|
bjori, is it actually an endless loop, or just three long retires? A fix via | |||||||||||||||||||||||
| Comment by Hannes Magnusson [ 07/Apr/15 ] | |||||||||||||||||||||||
|
It looks like this commit broke something. When I attempt to connect to a non-existing-node I wind up in an endless loop
And when SSL (hostname) verification fails:
Where the phongo_stream_initiator() returns NULL every single time. Note that thing that looks like a connection uri in the log – it is not – so take the options with grain of salt | |||||||||||||||||||||||
| Comment by Githook User [ 07/Apr/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@emptysquare.net'}Message: Merge pull request #213 from hanumantmk/
| |||||||||||||||||||||||
| Comment by Githook User [ 07/Apr/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@emptysquare.net'}Message: Merge pull request #213 from hanumantmk/
| |||||||||||||||||||||||
| Comment by Githook User [ 07/Apr/15 ] | |||||||||||||||||||||||
|
Author: {u'username': u'hanumantmk', u'name': u'Jason Carey (hanumantmk)', u'email': u'jcarey@argv.me'}Message: |