-
Type:
Bug
-
Resolution: Unresolved
-
Priority:
Major - P3
-
None
-
Affects Version/s: None
-
Component/s: APIs
-
Storage Engines
-
StorEng - Defined Pipeline
This was noticed by max.hirschhorn@mongodb.com with a followup by wei.hu@mongodb.com .
Flags are set here and here, and that is done when cursor debug copy is NOT set on the session. So it doesn't make sense. We can remove those lines IMHO.
The bug should be benign as far as correctness goes (i.e. I don't think a test can be constructed to trigger a problem), because these flags are not looked at unless debug copy IS set on the session. Still it's wrong, confusing, and will make the check in __cursor_copy_release a tiny bit slower.
We should take the opportunity to examine the use of these flags to make sure we haven't missed something else that was intended.