[SERVER-27042] Fix and modernize currentop.js Created: 15/Nov/16  Updated: 05/Apr/17  Resolved: 07/Dec/16

Status: Closed
Project: Core Server
Component/s: Querying
Affects Version/s: None
Fix Version/s: 3.2.12, 3.4.2, 3.5.1

Type: Improvement Priority: Major - P3
Reporter: Charlie Swanson Assignee: Charlie Swanson
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Depends
Duplicate
is duplicated by SERVER-26730 Rewrite currentop.js to use the setYi... Closed
Backwards Compatibility: Fully Compatible
Backport Requested:
v3.4, v3.2
Sprint: Query 2016-11-21, Query 2016-12-12
Participants:
Linked BF Score: 0

 Description   

The test doesn't properly wait for the operations to be running when using the mmapv1 storage engine. It looks for operations to report holding or waiting for the "w" or "r" collection locks, which represent intent locks. This won't happen on mmapv1. Instead, the operations will report holding or waiting for the "W" or "R" collection locks respectively (the non-intent versions of these locks, which are not compatible with each other).

While fixing the test, we should clean up the numerous unnecessary print statements, single letter variable names, etc. which make the test difficult to read and diagnose.



 Comments   
Comment by Githook User [ 22/Dec/16 ]

Author:

{u'username': u'cswanson310', u'name': u'Charlie Swanson', u'email': u'cswanson310@gmail.com'}

Message: SERVER-27042 Stabilize currentop.js and add unit tests for lock state reporting

(cherry picked from commit 0bca5d9fc70fe4178441eaf097324e6da814410a)
Branch: v3.4
https://github.com/mongodb/mongo/commit/0443f495ddf3f54ec1b9eb91718935ff20cd01f1

Comment by Githook User [ 13/Dec/16 ]

Author:

{u'username': u'cswanson310', u'name': u'Charlie Swanson', u'email': u'cswanson310@gmail.com'}

Message: SERVER-27042 Stabilize currentop.js and add unit tests for lock state reporting
Branch: v3.2
https://github.com/mongodb/mongo/commit/9d920663421437e65706b0587077781895984e2d

Comment by Githook User [ 13/Dec/16 ]

Author:

{u'username': u'cswanson310', u'name': u'Charlie Swanson', u'email': u'charlie.swanson@mongodb.com'}

Message: Revert "SERVER-27042 Stabilize currentop.js and add unit tests for lock state reporting"

This reverts commit 68bf5e7dddcc17b4fce0e5c602ad0f1bb423324d.
Branch: v3.4
https://github.com/mongodb/mongo/commit/63fa79b30347475962f1ca7ba8858c57baf33925

Comment by Githook User [ 13/Dec/16 ]

Author:

{u'username': u'cswanson310', u'name': u'Charlie Swanson', u'email': u'cswanson310@gmail.com'}

Message: SERVER-27042 Stabilize currentop.js and add unit tests for lock state reporting

(cherry picked from commit 0bca5d9fc70fe4178441eaf097324e6da814410a)
Branch: v3.4
https://github.com/mongodb/mongo/commit/68bf5e7dddcc17b4fce0e5c602ad0f1bb423324d

Comment by Githook User [ 07/Dec/16 ]

Author:

{u'username': u'cswanson310', u'name': u'Charlie Swanson', u'email': u'cswanson310@gmail.com'}

Message: SERVER-27042 Stabilize currentop.js and add unit tests for lock state reporting
Branch: master
https://github.com/mongodb/mongo/commit/0bca5d9fc70fe4178441eaf097324e6da814410a

Generated at Thu Feb 08 04:14:00 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.