[SERVER-21852] kill_cursors.js fails in small_oplog* configurations Created: 09/Dec/15 Updated: 16/Nov/16 Resolved: 11/Dec/15 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Querying |
| Affects Version/s: | None |
| Fix Version/s: | 3.2.1, 3.3.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | David Storch | Assignee: | David Storch |
| Resolution: | Done | Votes: | 0 |
| Labels: | test-only | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Operating System: | ALL |
| Backport Completed: | |
| Sprint: | QuInt D (12/14/15) |
| Participants: |
| Description |
|
| Comments |
| Comment by Githook User [ 11/Dec/15 ] | |||
|
Author: {u'username': u'dstorch', u'name': u'David Storch', u'email': u'david.storch@10gen.com'}Message: (cherry picked from commit 0ce05c7d8c4a7b309c9792d4350db680bdd7096d) | |||
| Comment by Githook User [ 11/Dec/15 ] | |||
|
Author: {u'username': u'dstorch', u'name': u'David Storch', u'email': u'david.storch@10gen.com'}Message: | |||
| Comment by Githook User [ 10/Dec/15 ] | |||
|
Author: {u'username': u'dstorch', u'name': u'David Storch', u'email': u'david.storch@10gen.com'}Message: (cherry picked from commit aa77f50235503a1c3c4883244ed9d805412c10fe) | |||
| Comment by Githook User [ 10/Dec/15 ] | |||
|
Author: {u'username': u'dstorch', u'name': u'David Storch', u'email': u'david.storch@10gen.com'}Message: | |||
| Comment by David Storch [ 10/Dec/15 ] | |||
|
The test works by enabling a failpoint that keeps a cursor pinned, and then running an assert.soon() which waits until the number of cursors pinned increases by 1:
In the master/slave configuration used by jsCore_small_oplog_WT, however, there ends up being an additional pinned cursor due to the slave syncing off the master. If this happens at the right time, the assert.soon() will never be satisfied because the current number of cursors will exceed numPinnedBefore by 2 rather than by 1. | |||
| Comment by Kyle Erf [ 10/Dec/15 ] | |||
|
Thanks! Just making sure that this wasn't getting lost in the handoff. Right now it's still only the small_oplog_* variants. | |||
| Comment by David Storch [ 10/Dec/15 ] | |||
|
I'll look at it, I just haven't yet. Have you seen it on anything other than small_oplog? | |||
| Comment by Kyle Erf [ 10/Dec/15 ] | |||
|
david.storch are you working on this, or does it still need triage/assignment? |