Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21684
Hm, I actually doubt it's ever correct to parse the result of
`getErrorStream` here as JSON. It's going to be text or HTML document. The
comment shows the intent was to return the error message to the user --
presumably as an exception as well, handled in the last line of the method. But
it won't get there, right? JSON parsing fails before.
Therefore, what about throwing an exception right away on any non-200
response? Sure, try to put the content of `getErrorStream` in the message.
`HistoryServerSuite` seems to do something like this.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
