Ngone51 commented on pull request #29468: URL: https://github.com/apache/spark/pull/29468#issuecomment-675917559
> Now all we need is a real test to validate that the PR is actually fixing the delay scheduling issue you pointed out. @agrawaldevesh I added a unit test for locality level computation in `TaskSetManagerSuite`. I'm not sure what do you mean by a "real test". But if it means an end to end test which validates this fix can avoid the unnecessary delay, then I think it can be very hard. ---------------------------------------------------------------- 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]
