wypoon commented on pull request #28848: URL: https://github.com/apache/spark/pull/28848#issuecomment-649874411
> @wypoon if you have not started extending the test with the multiple fetch failures case you can use this I you agree with it: > [attilapiros@be14a51](https://github.com/attilapiros/spark/commit/be14a51ca766711d793d9a7314a2cf030e2acdc7) @attilapiros thanks for the code; that is very helpful. I had an offline chat with @squito, and he had a different test in mind, but in a similar spirit. He was thinking of a test to verify that in `DAGScheduler`, `blockManagerMaster.removeExecutor` is not called more than once after the executor is lost. I can use your approach (using Mockito spy) there as well. ---------------------------------------------------------------- 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]
