Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/21577#discussion_r196247047 --- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala --- @@ -399,7 +399,8 @@ private[spark] object JsonProtocol { ("Full Stack Trace" -> exceptionFailure.fullStackTrace) ~ ("Accumulator Updates" -> accumUpdates) case taskCommitDenied: TaskCommitDenied => - ("Job ID" -> taskCommitDenied.jobID) ~ + ("Job ID" -> taskCommitDenied.stageID) ~ + ("Job Attempt Number" -> taskCommitDenied.stageAttempt) ~ --- End diff -- And it shouldn't affect compatibility. Given the code, even an old history server would be able to read these new log files.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org