[SERVER-58193] Add test coverage for $in matching against +/-NaN Created: 01/Jul/21 Updated: 29/Oct/23 Resolved: 24/Aug/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | 5.0.0-rc6 |
| Fix Version/s: | 5.1.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Mohammad Dashti (Inactive) | Assignee: | Mihai Andrei |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | sbe-diff, sbe-rollout | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Sprint: | QE 2021-08-23, QE 2021-09-06 | ||||||||
| Participants: | |||||||||
| Linked BF Score: | 28 | ||||||||
| Description |
|
Here are the steps to reproduce the issue:
As you see, the output of SBE has fewer elements compared to the classic engine, which hints into the direction that the special values (e.g., NaN and non-existing fields) accessed from SBE by $graphLookup are treated differently compared to the classic engine. |
| Comments |
| Comment by Vivian Ge (Inactive) [ 06/Oct/21 ] |
|
Updating the fixversion since branching activities occurred yesterday. This ticket will be in rc0 when it’s been triggered. For more active release information, please keep an eye on #server-release. Thank you! |
| Comment by Githook User [ 24/Aug/21 ] |
|
Author: {'name': 'Mihai Andrei', 'email': 'mihai.andrei@10gen.com', 'username': 'mtandrei'}Message: |
| Comment by Steve La (Inactive) [ 19/Aug/21 ] |
|
mihai.andrei. nice work! do we have any automated deterministic test ( not fuzzer ) that covers this case? If not, can we convert your repro into such a test?
|
| Comment by Mohammad Dashti (Inactive) [ 18/Aug/21 ] |
|
mihai.andrei this is a typo. The command should be the same for both engines. As a final check, can you see if with and without $limit: 20, the issue does not appear? |
| Comment by Steve La (Inactive) [ 17/Aug/21 ] |
|
thx mohammad.dashti i pasted the below comment in the other similar ticket. https://jira.mongodb.org/browse/SERVER-58155 mihai.andrei. i see . this goes towards my suspicion that this is not specific to lookup or graphLookup ( i commented on another similar ticket for that mihai.andrei mohammad.dashti could one of you look to try both out for simple match? this will allow use to potentially dedup these 2 problems to 1 that would help us to better understand what work we have left kyle.suarez FYI
|
| Comment by Mohammad Dashti (Inactive) [ 17/Aug/21 ] |
|
steve.la Even though $graphLookup itself is not supported by SBE, the rest of the query execution pipeline uses SBE. In the above example, the part that's handled by SBE is different from its equivalent in the classic engine. It's possible to recreate this issue without $graphLookup, but used the above example, which stems from BF-21717. |