Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/3816#issuecomment-68282936
Why was this a problem? You need to make sure that this won't change the
locality level the scheduler launches tasks at due to delay scheduling. For
example, if a stage contained both process-local and no-pref tasks, and it was
still able to launch tasks locally (without the delay expiring), this change
might make it forget that and not wait long enough, thus not getting local
tasks. Please write down something explaining why this was a problem and why
the fix won't break other things.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
