skyplane-project / skyplane

🔥 Blazing fast bulk data transfers between any cloud 🔥
https://skyplane.org
Apache License 2.0
1.09k stars 62 forks source link

[CLI] Merge `skyplane cp` and `skypane sync` implementation #550

Closed ShishirPatil closed 2 years ago

ShishirPatil commented 2 years ago

The right design would be - Skyplane sync should draw up a list of chunks to transfer from source bucket to destination bucket and then call skyplane cp. A lot of duplicated code as of now.

parasj commented 2 years ago

I've minimized as much of the duplicated code as I can. What's left is the public CLI interface which will need to remain regardless of what we do. With #591, we can further eliminate duplicated code. Closing this for now.