Update the error message not to mention cancelling, as the sendResult() method is used for sending the completion intent as well, and also include the exception stack trace.
[x] I ran, updated and added unit tests as necessary.
[x] I verified the contribution matches existing coding style.
[x] I updated the documentation if necessary.
Motivation and Context
The sendResult() method is used for sending the completion intent as well as the cancellation intent, but its error message only mentions cancellation, which can be confusing. In addition, the additional exception parameter passed to Logger.error() is ignored.
Description
Update the error message not to mention cancelling, and also include the exception stack trace.
sheepmaster
commented
3 months ago
Update the error message not to mention cancelling, as the
sendResult()method is used for sending the completion intent as well, and also include the exception stack trace.Checklist
Motivation and Context
The
sendResult()method is used for sending the completion intent as well as the cancellation intent, but its error message only mentions cancellation, which can be confusing. In addition, the additional exception parameter passed toLogger.error()is ignored.Description
Update the error message not to mention cancelling, and also include the exception stack trace.