[SERVER-61385] Migrate most callers of legacy 'DBClientBase::query()' API to modern 'find()' API Created: 10/Nov/21 Updated: 29/Oct/23 Resolved: 21/Dec/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Internal Client |
| Affects Version/s: | None |
| Fix Version/s: | 5.3.0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | David Storch | Assignee: | David Storch |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||
| Sprint: | QE 2021-11-29, QE 2021-12-13, QE 2021-12-27 | ||||||||||||||||||||
| Participants: | |||||||||||||||||||||
| Linked BF Score: | 64 | ||||||||||||||||||||
| Description |
|
Related ticket The goal of this ticket is to move callers of the legacy DBClientBase::query() API onto the new DBClientBase::find() API. Once this migration is complete, the code which implements DBClientBase::query(), including the special upconversion logic mentioned above, should be deleted. This may need to be done in multiple steps, depending on the complexity and the scope of the code changes. |
| Comments |
| Comment by Githook User [ 23/Dec/21 ] |
|
Author: {'name': 'David Storch', 'email': 'david.storch@mongodb.com', 'username': 'dstorch'}Message: |
| Comment by David Storch [ 21/Dec/21 ] |
|
Due to the scope of the changes required to fully eliminate the legacy DBClientBase::query() API from the code base, we now intend to do it in two steps. The changes committed under this ticket moved most but not all callers to the new API. It also renamed the old API to query_DEPRECATED() in order to discourage adding more uses in the future. I've filed followup ticket |
| Comment by Githook User [ 21/Dec/21 ] |
|
Author: {'name': 'David Storch', 'email': 'david.storch@mongodb.com', 'username': 'dstorch'}Message: There are a handful of remaining callers of the legacy API, |