attilapiros edited a comment on pull request #31513:
URL: https://github.com/apache/spark/pull/31513#issuecomment-785248802
I come up with a good test (but my work is still ongoing):
```
// N-: newly created not known by the scheduler backend
// N+: newly created scheduler backend known
// P- / P+ : pending
// D: deleted
// | default || rp
|
// | ||
|
// | 1 | 2 | 3 || 4 | 5 | 6 |
7 |
//--------------------------------------------------------------------------
// 0) setTotalExpectedExecs with | N- | N- | N- || N- | N- | N- |
N- | so
// default->3, ro->4 | | | || | | |
| outstanding == 7
//---------------------------------------------------------------------------
// 1) make 1 from each rp | N+ | N- | N- || N+ | N- | N- |
N- |
// known by backend | | | || | | |
| outstanding == 5
//---------------------------------------------------------------------------
// 2) some more backend known + pending | N+ | P+ | P- || N+ | P+ | P- |
N- | outstanding == 3
//---------------------------------------------------------------------------
// 3) advance time with idle timeout | | | || | | |
|
// setTotalExpectedExecs with | N+ | P+ | D || N+ | P+ | D |
D | outstanding == 0
// default->1, rp->1 | | | || | | |
|
//---------------------------------------------------------------------------
// 4) setTotalExpectedExecs with | N+ | P+ | D || N+ | P+ | D |
D | outstanding == 0
// default->2, rp->2 | | | || | | |
| no new POD req.
//
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
