search

ECP-VeloC / AXL

Asynchronous Transfer Library
MIT License
2 stars 8 forks source link

BBAPI fallback to sync if FS doesn't support extents #62

Closed tonyhutter closed 4 years ago

tonyhutter commented 4 years ago

If the user requests a BBAPI transfer, and one or more of the source or destination paths in the file list aren't to a filesystem that BBAPI supports, then fallback to doing a pthreads transfer for all the files in the file list. This gets around BBAPI giving you a "file not found" when it tries to transfer to/from a filesystem that doesn't support extents (like NFS).

Fixes: #61

tonyhutter commented 4 years ago

Reviewers: should this fall back to pthreads instead of sync?

adammoody commented 4 years ago

I'm fine with having this fallback to pthreads if you want. Seems like that's closer to what the user might expect if they want this to be async.

I'm also fine with leaving the fallback as sync, so your call.

tonyhutter commented 4 years ago

Ok, I'm going to update it to use pthreads then. I'll add in your other changes too.

tonyhutter commented 4 years ago

@gonsie @adammoody my latest push addresses all your comments and falls back to pthreads instead of sync.

adammoody commented 4 years ago

Thanks @tonyhutter .