[SERVER-84317] Use named args for clarity in queryStats JS assert API calls Created: 19/Dec/23 Updated: 04/Jan/24 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | William Qian | Assignee: | Backlog - Query Integration |
| Resolution: | Unresolved | Votes: | 1 |
| Labels: | neweng, qi-query-stats, quick-tech-debt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Query Integration
|
| Participants: |
| Description |
|
Query stats utils currently uses a position-based argument system, which is annoying to document and prone to unnecessary verbosity. Passing in a parameter object would make this kind of code much more readable and rely less on the weird commenting antipattern, such as here: here. |
| Comments |
| Comment by Charlie Swanson [ 20/Dec/23 ] |
|
Kicking this out of PM-2885 epic since we are bringing that to a close. I'll flag for normal triage by QI - likely a good "quick-tech-debt" and "neweng" ticket. |