You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Split up Simple Staging and Legacy Staging Logic (#1022)
Summary:
Pull Request resolved: #1022
Currently base_async_storage_writer is quite complicated and has a bunch of branching between simple_staging and legacy staging. We are looking to introduce yet another simple staging solution. So refactor as follows:
Last half, we introduced stager as an argument to DCP async_save.
1. Introduce LegacyStager and move legacy staging logic intop it
2. Introduce SimpleStager and move simple staging logic into it.
3. Based on JKs in checkpoint_dist_client initialize the right stager to use.
4. Remove Staging Logic From BaseAsyncStorageWriter
Differential Revision: D79434609
0 commit comments