Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21577#discussion_r196246952
--- 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 --
For the new property, I'm just following what the old property says, even
though it's wrong. I think having `Job ID` and `Stage Attempt Number` would
just be even more confusing...
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
