mridulm commented on pull request #33872: URL: https://github.com/apache/spark/pull/33872#issuecomment-914630622
@sleep1661 The test is not surfacing an issue in master - can you reformulate it such that the problem is surfaced in master without the patch and gets fixed after the patch ? (Currently it is failing due to `tasksSuccessful` > 0 - which is not an error in itself). @Ngone51 I was initially looking at the patch from point of view of inconsistent events being fired - (two events for the same task - first failed and then success) : and from that point of view, it looked like a right fix to have. It is a good idea to investigate if the root cause reported is actually an issue - namely if we hang. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
