[SERVER-29091] Stop putting $readPreference inside of $queryOptions in mongos Created: 05/May/17 Updated: 30/Jan/24 |
|
| Status: | Open |
| Project: | Core Server |
| Component/s: | Internal Code, Sharding |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Mathias Stearn | Assignee: | Backlog - Query Execution |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | PM-315 | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Assigned Teams: |
Query Execution
|
||||||||
| Sprint: | QE 2023-02-06, QE 2023-02-20, QE 2023-07-24, QE 2023-08-07, QE 2023-08-21, QE 2023-09-04 | ||||||||
| Participants: | |||||||||
| Description |
|
Mongos currently wraps the $readPreference field inside of a $queryOptions object. Instead it should just put $readPreference at the top-level of the command, matching the format used by OP_MSG. |
| Comments |
| Comment by David Storch [ 22/Feb/23 ] |
|
I don't think I'm going to get to this given my current queue of tickets. We could consider it as a quick win nomination, so I'll request it to be re-triaged. |
| Comment by David Storch [ 13/Dec/22 ] |
|
brenda.rodriguez@mongodb.com I've looked at this a bit already and it wasn't as simple I was hoping. However, I did complete related ticket |
| Comment by Brenda Rodriguez [ 12/Dec/22 ] |
|
david.storch@mongodb.com are you still planning to do this during this sprint? |
| Comment by David Storch [ 16/Nov/22 ] |
|
I looked into this a bit and found that there are still quite a few uses of "$queryOptions" in the code base. It also looks like we might be sending "$queryOptions" over the wire between nodes in a few cases which, if true, would require the work to be staged across multiple LTS releases. I also concluded that it is in fact easier to do |
| Comment by David Storch [ 17/Oct/22 ] |
|
I think this should be a lot easier to do now that the OP_QUERY wire protocol message is no longer supported. I'm also wondering if doing this will make |
| Comment by Mathias Stearn [ 03/May/18 ] |
|
| Comment by Gregory McKeon (Inactive) [ 13/Nov/17 ] |
|
redbeard0531 can you clarify what needs to be done here? |