tgravescs commented on issue #27207: [WIP][SPARK-18886][CORE] Make Locality 
wait time measure resource under utilization due to delay scheduling.
URL: https://github.com/apache/spark/pull/27207#issuecomment-577746374
 
 
   So the conditions would be like:
   
   offer 1 resource that was rejected - no timer reset
   offer all resources with no rejects and launched a task - timer is reset
   offer 1 resource, no reject - timer is reset
   offer 1 resource that was rejected - no timer reset
   offer 1 resource, no reject - no timer reset because previous offer was 
rejected
   
   I think that makes sense, it definitely addresses the case I was talking 
about with 2 task sets.
   
   I was also wondering about leaving the old logic in there but configured off 
by default.  While I don't like it, it would be safer if we haven't thought of 
some corner case and user could turn it back on if necessary, thoughts?

----------------------------------------------------------------
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]

Reply via email to