search

DLHub-Argonne / dlhub_sdk

Python Interface to the Data and Learning Hub for Science
Apache License 2.0
27 stars 6 forks source link

Replace DLHubFuture with FuncXFuture #93

Closed WardLT closed 3 years ago

WardLT commented 4 years ago

I do not believe the DLHubFuture is used by the latest version of the client. We should replace it with the FuncX version of a Future for task results

WardLT commented 4 years ago

Correction: The DLHubFuture is used, but still seems superfluous

WardLT commented 3 years ago

Given the special logic introduced in #127 for when to return invocation metadata or not, we will still need the DLHubFuture for the time being