Details
-
Bug
-
Resolution: Fixed
-
Major
-
Cheshire-Cat
-
Untriaged
-
1
-
Unknown
Description
Although we don't like values to be used concurrently, there's a couple of places where this happens (unnest being one).
We have a mutex which is used for concurrent Field() and Index() access.
Snag is, unwrap clears fields and elements without mutex which could potentially lead to concurrent map reads and writes.
We have no such case in query, but one suspected case in the projector, so it's probably best to add concurrency control in unwrap().
Attachments
For Gerrit Dashboard: MB-44144 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
145679,4 | MB-44144 free parsed fields and elements under a lock in parsed.unwrap() | mad-hatter | query | Status: MERGED | +2 | +1 |
145700,3 | MB-44144 free parsed fields and elements under a lock in parsed.unwrap() | master | query | Status: MERGED | +2 | +1 |