Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17882#discussion_r120809352
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnSchedulerBackend.scala
---
@@ -176,16 +179,6 @@ private[spark]
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17882#discussion_r120710864
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnSchedulerBackend.scala
---
@@ -307,6 +301,9 @@ private[spark]
Github user witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/17882#discussion_r120652302
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnSchedulerBackend.scala
---
@@ -68,6 +68,8 @@ private[spark] abstract
Github user witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/17882#discussion_r120644588
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnSchedulerBackend.scala
---
@@ -176,16 +179,6 @@ private[spark]