kuwii commented on PR #43646: URL: https://github.com/apache/spark/pull/43646#issuecomment-1826186284
> Seems OK. It's a little bit better than the raw stack trace. I don't suppose there's a similar change to make in the main Spark UI? or does it also already work this way? I don't think Spark's built-in UI needs this change, because web page doesn't seem to throw `WebApplicationException`, and REST API already supports this. The gap may be because web page and REST API are using different ways to handle HTTP requests. The main goal of this PR is to let web page part also able to return customized response code (like 418 as an example), instead of just 200, 400, 404 and 500. It is basically for those who wants to extend Spark UI through plugins, without forking Spark source code. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
