futureverse / futureverse/parallelly
Add forknode method for isNodeAlive
- 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
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