Address tgraves review comments

Author: venkata91

core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala

@@ -882,13 +882,6 @@ private[spark] class ExecutorAllocationManager(
       attempts.filter(attempt => unschedulableTaskSets.contains(attempt)).size
     }
 
-    def hasPendingUnschedulableTasks: Boolean = {
-      val attemptSets = resourceProfileIdToStageAttempt.values
-      attemptSets.exists { attempts =>
-        attempts.exists(unschedulableTaskSets.contains(_))
-      }
-    }
-
     def hasPendingTasks: Boolean = {
       hasPendingSpeculativeTasks || hasPendingRegularTasks
     }

core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala

@@ -1041,8 +1041,8 @@ private[spark] class DAGScheduler(
   }
 
   private[scheduler] def handleUnschedulableTaskSetRemoved(
-    stageId: Int,
-    stageAttemptId: Int): Unit = {
+      stageId: Int,
+      stageAttemptId: Int): Unit = {
     listenerBus.post(SparkListenerUnschedulableTaskSetRemoved(stageId, stageAttemptId))
   }