[
https://issues.apache.org/jira/browse/YARN-11893?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18056450#comment-18056450
]
ASF GitHub Bot commented on YARN-11893:
---------------------------------------
p-szucs commented on PR #8066:
URL: https://github.com/apache/hadoop/pull/8066#issuecomment-3847800222
Thank you for the review @K0K0V0K, @slfan1989!
The reason I originally kept these changes in one PR because the only
functional change is in YARN common's GenericExceptionHandler, and just wanted
to align all the related unit tests in one go, to avoid breaking them e.g. in a
later backport. But please let me know if you still think it would be better in
two splits
> Extend REST API error messages with exception classes
> -----------------------------------------------------
>
> Key: YARN-11893
> URL: https://issues.apache.org/jira/browse/YARN-11893
> Project: Hadoop YARN
> Issue Type: Improvement
> Components: yarn, yarn-common
> Affects Versions: 3.5.0
> Reporter: Peter Szucs
> Assignee: Peter Szucs
> Priority: Minor
> Labels: pull-request-available
>
> In GenericExceptionHandler's toResponse method we put only the exception
> message into the RemoteExceptionData object.
> It would be a bit more informative if we would use throwable.toString()
> instead of throwable.getMessage() as the former extends the error message
> with the exception class as well, similarly to previous versions.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
