[SERVER-58380] Pass arguments in performUpdates, performInserts, and performDeletes by value Created: 08/Jul/21 Updated: 29/Oct/23 Resolved: 12/Aug/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 5.1.0-rc0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Kris Satya (Inactive) | Assignee: | Kris Satya (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Sprint: | Execution Team 2021-08-23 |
| Participants: |
| Description |
|
PerformInserts, performUpdates, and performDeletes all accept OperationSource as a parameter by const reference. However, there is really no need to pass by const ref since OperationSource is an enum, so we should modify these functions to pass their parameters by value instead. |
| Comments |
| Comment by Vivian Ge (Inactive) [ 06/Oct/21 ] |
|
Updating the fixversion since branching activities occurred yesterday. This ticket will be in rc0 when it’s been triggered. For more active release information, please keep an eye on #server-release. Thank you! |
| Comment by Githook User [ 12/Aug/21 ] |
|
Author: {'name': 'Kris Satya', 'email': 'kris.satya@mongodb.com', 'username': 'KoolCards'}Message: |