bmarcott commented on issue #28188: [SPARK-18886][CORE][FOLLOWUP] remove requirement to launch a task to reset locality wait timer URL: https://github.com/apache/spark/pull/28188#issuecomment-613223634 Meanwhile.. I added a test case which matches that scenario you described in the other PR. I changed the code such that launching a task is still required to reset the timer, but not launching a task does not prevent you from resetting on a follow up offer. This passes the new test and previous ones. It seems better to remove the requirement to launch a task, but I'm ok with this since it isn't worse than legacy locality wait.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
