Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19383#discussion_r147766895
--- Diff: project/MimaExcludes.scala ---
@@ -36,6 +36,8 @@ object MimaExcludes {
// Exclude rules for 2.3.x
lazy val v23excludes = v22excludes ++ Seq(
+ // SPARK-18085: Better History Server scalability for many / large
applications
+
ProblemFilters.exclude[IncompatibleResultTypeProblem]("org.apache.spark.status.api.v1.ExecutorSummary.executorLogs"),
--- End diff --
Well, it doesn't break compatibility because this class is not to be used
outside Spark (it's only public so mima can detect breakages in the JSON
format, and this doesn't change that).
Leaving the old import makes using this class really awkward from other
places that don't have that import.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
