[SERVER-83550] Remove duplicate helper functions related to backup cursors Created: 22/Nov/23  Updated: 10/Jan/24  Resolved: 10/Jan/24

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Major - P3
Reporter: Etienne Petrel Assignee: Gregory Noma
Resolution: Won't Do Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Gantt Dependency
has to be done after SERVER-83477 Add helper functions for backup curso... Closed
Assigned Teams:
Storage Execution
Sprint: Execution Team 2024-01-22
Participants:

 Description   

With SERVER-83477, jstests/libs/backup_utils.js was updated with improved utility functions to open backup cursors.
Similar functions are present in the following enterprise folder: jstests/hot_backups/libs. If possible, we should just remove the duplicate. As the backup tests are only in enterprise, we could move everything in there?



 Comments   
Comment by Gregory Noma [ 10/Jan/24 ]

After looking into this a bit, I'm actually not sure if this is worth doing. One file contains the logic for incremental backups and the other is for non-incremental backups. In theory we could still combine them since the logic is similar, but it may make the code more complicated and the function purpose bloated. I think there are pros and cons both ways so I'd lean toward leaving it as-is unless something changes in the future.

Generated at Thu Feb 08 06:52:31 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.