[SERVER-72068] CollectionQueryInfo::notifyOfQuery doesn't use any state from CollectionQueryInfo Created: 13/Dec/22 Updated: 13/Feb/23 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Matthew Saltz (Inactive) | Assignee: | Backlog - Storage Execution Team |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | neweng, techdebt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Storage Execution
|
| Participants: |
| Description |
|
CollectionQueryInfo::notifyOfQuery is a non-static member function of CollectionQueryInfo, but it doesn't use any state from that class. It actually just gets an instance of CollectionIndexUsageTrackerDecoration from one of the parameters that was passed in, and calls a few member functions on it. It seems like there could instead just be a function CollectionIndexUsageTrackerDecoration::notifyOfQuery, or a free function somewhere notifyCollectionIndexUsageTrackerOfQuery - unless there's some reason this has to be on the CollectionQueryInfo |
| Comments |
| Comment by Steve Tarzia [ 13/Dec/22 ] |
|
Assigning to StorEx as it related to the work happening there recently. |