- 
    Type:Task 
- 
    Resolution: Unresolved
- 
    Priority:Major - P3 
- 
    None
- 
    Affects Version/s: None
- 
    Component/s: None
- 
    None
- 
        Storage Execution
- 
        Execution Team 2024-03-04, Execution Team 2024-03-18, Execution Team 2024-04-01, Execution Team 2024-06-10, Execution Team 2024-06-24, Execution Team 2024-07-08, Execution Team 2024-07-22
- 
        None
- 
        None
- 
        None
- 
        None
- 
        None
- 
        None
- 
        None
Currently there is code in insertDocumentForBulkLoader that checks for when replCoord->isOplogDisabledFor(...) returns false, here.
I looked for callers of the insertDocumentForBulkLoader function and it seems it's only called from CollectionCloner, which is only used in initial sync. So that means that there won't ever be a case where isOplogDisabledFor returns false, as the oplog is disabled during initial sync.
A patch with the following added to it ran successfully:
invariant(replCoord->isOplogDisabledFor(opCtx, nss));
So we should be able to simplify that function since we are sure that it's only called in the initial sync context (for example, we can remove the check linked above).