HeartSaVioR commented on a change in pull request #23790: [SPARK-26792][CORE] 
Apply custom log URL to Spark UI
URL: https://github.com/apache/spark/pull/23790#discussion_r257008695
 
 

 ##########
 File path: 
core/src/test/scala/org/apache/spark/scheduler/CoarseGrainedSchedulerBackendSuite.scala
 ##########
 @@ -96,6 +100,10 @@ class CoarseGrainedSchedulerBackendSuite extends 
SparkFunSuite with LocalSparkCo
       override def onTaskEnd(taskEnd: SparkListenerTaskEnd): Unit = {
         taskEnded.set(true)
       }
+
+      override def onExecutorAdded(executorAdded: SparkListenerExecutorAdded): 
Unit = {
 
 Review comment:
   Ah yes nice finding. I was about to add code there to test but changed my 
mind. It should be removed. Will remove.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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