Refactor `Gitlab::Geo::Replication::BaseTransfer`
The following discussion from !15606 (merged) should be addressed:
-
@mkozono started a discussion: An idea/nitpick: Lack of
filenameimplies remote store. We are depending on this meaning in this test and its corresponding condition incan_transfer?.We could explicitly initialize
BaseTransferwith e.g.remote_store:. This would also allow the Downloaders to call e.g.transfer.execute, andtransferwould know whether to download or stream to object storage.This could be done in a follow up, if desired.