Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/22165
@jiangxb1987 Great thanks for your comment!
```
One general idea is that we don't need to rely on the RPC framework to test
ContextBarrierState, just mock RpcCallContexts should be enough.
```
Actually I also want to implement like this at first also as you asked in
jira, but `ContextBarrierState` is the private inner class in
`BarrierCoordinator`. Could I do the refactor of moving `ContextBarrierState`
out of `BarrierCoordinator`? If that is permitted I think we can just mock
RpcCallContext to reach this.
```
We shall cover the following scenarios:
```
Pretty cool for the list, the 5 in front scenarios are including in
currently implement, I'll add the last checking work of `Make sure we clear all
the internal data under each case.` after we reach an agreement.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
