[SERVER-55554] Fix out of order array element updates for operators like $min Created: 26/Mar/21 Updated: 19/May/21 Resolved: 19/May/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | 4.4.4 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Arun Banala | Assignee: | James Wahlin |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Operating System: | ALL | ||||||||||||||||
| Sprint: | Query Optimization 2021-05-31 | ||||||||||||||||
| Participants: | |||||||||||||||||
| Description |
|
This issue was fixed on master by |
| Comments |
| Comment by James Wahlin [ 19/May/21 ] | |||||||||||||||
|
Closing as a duplicate of | |||||||||||||||
| Comment by James Wahlin [ 19/May/21 ] | |||||||||||||||
|
While a backport to 4.4 of | |||||||||||||||
| Comment by Ana Meza [ 08/Apr/21 ] | |||||||||||||||
|
Moving this to QO, we think that the fix for this is to Back Port | |||||||||||||||
| Comment by Sam Mercier [ 31/Mar/21 ] | |||||||||||||||
|
Ohhhhh we do on 4.4. Okay gotcha. Sorry that was the bit I was missing. Thanks for being patient with me arun.banala. Sorry I didn't have more to contribute. | |||||||||||||||
| Comment by Arun Banala [ 31/Mar/21 ] | |||||||||||||||
|
samuel.mercier For the case on $min, we don't actually error. We end up doing wrong update operation, which is a bigger problem than just failing, since it's misleading the users. I think we should fix the problem or at least convert this to an error on 4.4. | |||||||||||||||
| Comment by Sam Mercier [ 31/Mar/21 ] | |||||||||||||||
|
Sorry for butting in y'all. But I think we already have precedent for ignoring such errors. (At least the errors that BF-20201 tracks). Let me know if I'm entirely misunderstanding the situation. See: | |||||||||||||||
| Comment by Kyle Suarez [ 26/Mar/21 ] | |||||||||||||||
|
In general, I think the newer behavior post- | |||||||||||||||
| Comment by Arun Banala [ 26/Mar/21 ] | |||||||||||||||
|
Backporting Repro.
|