[SERVER-84743] [CQF] Consolidate runWithParams and runWithParamsAllNodes Created: 10/Jan/24 Updated: 01/Feb/24 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Militsa Sotirova | Assignee: | Backlog - Query Optimization |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | techdebt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Query Optimization
|
| Participants: |
| Description |
|
runWithParams in optimizer_utils.js only works for standalone db passthrough tests (it references db, which is not a parameter). In I was talking to david.percy@mongodb.com about this and he suggested an approach where we introduce a global variable that keeps track of whether we should actually print the output, add logic in golden_test.js and golden_overrides.js to add an override for setParameterOnAllHosts. This override would use set the global variable to disable printing for the function.
This ticket tracks the work to implement the above and consolidate the two runWithParams functions. |