mridulm commented on pull request #29113: URL: https://github.com/apache/spark/pull/29113#issuecomment-735208487
To clarify what I meant: 1. `ExceptionFailure` is a public api. 2. History server is not the only consumer of event files. 3. There are code paths which are populating `stackTrace` in `ExceptionFailure` Given all of these, setting stack trace to empty is backwardly incompatible against current spark versions, and breaks compatibility. I would have been in favour of the change if (3) above was not the case. Given all of these three conditions currently applying, this is a breaking change and I am not in favor of it. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
