[GitHub] spark issue #19812: [SPARK-22598][CORE] ExecutorAllocationManager does not r...

2017-11-29 Thread liutang123
Github user liutang123 commented on the issue:

https://github.com/apache/spark/pull/19812
  
Sorry, I can not reproduce it now. But, sometimes, 
`ExecutorAllocationManager ` did not request new executors and 
`YarnSchedulerBackend.requestedTotalExecutors` is 0. I will close this PR now 
and reopen it next time when I encounter this situation. 


---

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



[GitHub] spark issue #19812: [SPARK-22598][CORE] ExecutorAllocationManager does not r...

2017-11-28 Thread jerryshao
Github user jerryshao commented on the issue:

https://github.com/apache/spark/pull/19812
  
Does this failure ". For some reason, all of the 3 executors failed. " 
happened during task running or before task submission? Besides, if you're 
running on yarn, yarn will bring new executors to meet the requirement, also 
`ExecutorAllocationManager` will be notified with executor lost/register. Can 
you please tell us how to reproduce your scenario?




---

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



[GitHub] spark issue #19812: [SPARK-22598][CORE] ExecutorAllocationManager does not r...

2017-11-28 Thread liutang123
Github user liutang123 commented on the issue:

https://github.com/apache/spark/pull/19812
  
Hi @jerryshao , I modified the info of this PR.


---

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



[GitHub] spark issue #19812: [SPARK-22598][CORE] ExecutorAllocationManager does not r...

2017-11-28 Thread jerryshao
Github user jerryshao commented on the issue:

https://github.com/apache/spark/pull/19812
  
Hi @liutang123 would you mind explaining us the issue you met and how to 
reproduce it? Currently we don't know what actual issue it is and how to 
evaluate your changes.


---

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



[GitHub] spark issue #19812: [SPARK-22598][CORE] ExecutorAllocationManager does not r...

2017-11-26 Thread liutang123
Github user liutang123 commented on the issue:

https://github.com/apache/spark/pull/19812
  
@srowen Would you please look at this when you have time?


---

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



[GitHub] spark issue #19812: [SPARK-22598][CORE] ExecutorAllocationManager does not r...

2017-11-24 Thread liutang123
Github user liutang123 commented on the issue:

https://github.com/apache/spark/pull/19812
  
Jenkins, retest this please


---

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



[GitHub] spark issue #19812: [SPARK-22598][CORE] ExecutorAllocationManager does not r...

2017-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/19812
  
Can one of the admins verify this patch?


---

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