futureverse / futureverse/parallelly

Add forknode method for isNodeAlive

Open
#143 2 comments 0 reactions 0 assignees View on GitHub
feature request
Dominant language
R
Stars
140
Forks
9
PR merge metrics
No merged PRs in 30d

Description

(Please use for Q&A)

**Wish or feature request**

`isNodeAlive()` currently returns `NA` no matter the state of the cluster, which is forkin' useless.

Suggest adding a method for `SOCK0node` which checks the connection status.

```
ccc <- makeForkCluster(2)
> isNodeAlive(ccc)
[1] NA NA
> isNodeAlive.SOCK0node <- function (x, ...) isConnectionValid(x$con)
> isNodeAlive(ccc)
[1] TRUE TRUE
> stopCluster(ccc)
> isNodeAlive(ccc)
[1] FALSE FALSE
>
```

See futureverse/parallelly#136

Contributor guide

Open the contributing guide

Research direction

Start by locating the SOCK0node implementation and the existing isNodeAlive() and isConnectionValid() methods. Check how makeForkCluster() and stopCluster() represent connection state, then verify that isNodeAlive() reports TRUE for live nodes and FALSE after stopping the cluster, as shown in the issue example.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
distributed-systems
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.