Github user smurakozi commented on a diff in the pull request: https://github.com/apache/spark/pull/19920#discussion_r155609528 --- Diff: core/src/main/scala/org/apache/spark/status/AppStatusListener.scala --- @@ -29,6 +29,7 @@ import org.apache.spark.executor.TaskMetrics import org.apache.spark.internal.Logging import org.apache.spark.scheduler._ import org.apache.spark.status.api.v1 +import org.apache.spark.status.api.v1.{ApplicationAttemptInfo, ApplicationEnvironmentInfo, ApplicationInfo, RuntimeInfo} --- End diff -- There is no ambiguity between ApplicationAttemptInfo and ApplicationAttempt classes in different versions anymore, so I thought the v1-s can be removed. I can undo that if they are still useful.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org