cloudpathlib icon indicating copy to clipboard operation
cloudpathlib copied to clipboard

Avoid making mtime checks if force_overwrite_to_cloud is set

Open Gilthans opened this issue 2 years ago • 0 comments

Hey there, I noticed that when uploading files, even if force_overwrite_to_cloud is set, local + cloud mtime of the file are still calculated, even though they are effectively ignored. Usually this is a minor performance overhead, but it can have a large impact when uploading a directory with many small files (like an image pyramid for example). Since this does not affect functionality at all, I think this is an easy win!

I'll submit a PR soon, LMK if I missed anything

Gilthans avatar Nov 07 '23 11:11 Gilthans