You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/03/05 03:22:16 UTC

[GitHub] [spark] Ngone51 commented on issue #23842: [SPARK-26927]Fix race condition may cause dynamic allocation not working

Ngone51 commented on issue #23842: [SPARK-26927]Fix race condition may cause dynamic allocation not working
URL: https://github.com/apache/spark/pull/23842#issuecomment-469521185
 
 
   While dropping events maybe less harmful to the application, it cannot change the fact that ExecutorAllocationManager would result in wrong behaviour in the end. Actually, I'm not sure what's the right way to follow when the corner case happens.  I think we need to listener to other people's suggestion.
   
   What is `executorManagement ` ? I do not find it in ExecutorAllocationManager.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org