[SERVER-75724] Break up large functions Tenant Oplog Applier Created: 05/Apr/23 Updated: 29/Oct/23 Resolved: 31/May/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 7.1.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Didier Nadeau | Assignee: | Christopher Caplinger |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | serverless-shortlist | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Serverless
|
| Backwards Compatibility: | Fully Compatible |
| Sprint: | Server Serverless 2023-05-29, Server Serverless 2023-06-12 |
| Participants: |
| Description |
|
Some methods in TenantOplogApplier are very long and difficult to navigate such as writeNoop. We need to keep TenantOplogApplier for Shard Merge so we should improve this method to split it into understandable methods. In this ticket we aim to improve readability by breaking up these large methods into smaller ones. We should also take note of other possible improvements in tenant oplog applier and raise tickets for them (unless these improvements are trivial and can be done in the same PR with negligible impact). |
| Comments |
| Comment by Githook User [ 24/May/23 ] |
|
Author: {'name': 'Christopher Caplinger', 'email': 'christopher.caplinger@mongodb.com', 'username': 'UnicodeSnowman'}Message: |