resolved() for ClusterFuture:s would produce Error: 'inherits(future,…

… "Future")' is not TRUE [#723]
HenrikBengtsson · May 28, 2024 · 99f8dda · 99f8dda
1 parent eaa6d48
commit 99f8dda
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,5 +1,5 @@
 Package: future
-Version: 1.33.2-9000
+Version: 1.33.2-9001
 Title: Unified Parallel and Distributed Processing in R for Everyone
 Imports:
     digest,

diff --git a/NEWS.md b/NEWS.md
@@ -1,6 +1,11 @@
 # Version (development version)
 
- * ...
+## Bug Fixes
+
+ * `resolved()` for `ClusterFuture`:s would produce `Error:
+   'inherits(future, "Future")' is not TRUE` instead of an intended,
+   informative error message that the connection to the parallel
+   worker is broken.
 
 
 # Version 1.33.2 [2024-03-23]

diff --git a/R/ClusterFuture-class.R b/R/ClusterFuture-class.R
@@ -280,8 +280,8 @@ resolved.ClusterFuture <- function(x, run = TRUE, timeout = NULL, ...) {
     isValid <- isConnectionValid(con)
     if (!isValid) {
       ex <- simpleError("Connection to the worker is corrupt")
-      msg <- post_mortem_cluster_failure(ex, when = "checking resolved from", node = node, future = future)
-      stop(FutureError(msg, future = future))
+      msg <- post_mortem_cluster_failure(ex, when = "checking resolved from", node = node, future = x)
+      stop(FutureError(msg, future = x))
     }
 
     if (is.null(timeout)) {