attilapiros commented on a change in pull request #22290: [SPARK-25285][CORE] Add startedTasks and finishedTasks to the metrics system in the executor instance URL: https://github.com/apache/spark/pull/22290#discussion_r273567244
########## File path: core/src/main/scala/org/apache/spark/executor/ExecutorSource.scala ########## @@ -95,7 +100,8 @@ class ExecutorSource(threadPool: ThreadPoolExecutor, executorId: String) extends }) // Expose executor task metrics using the Dropwizard metrics system. - // The list is taken from TaskMetrics.scala + // The list of available Task metrics is available in TaskMetrics.scala Review comment: Nit: This sentence is strange, because of the two available words: "available Task metrics is available". ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
