Integrated: Log warning on wrapped UncheckedRestException

Erik Joelsson erikj at openjdk.java.net
Mon Feb 28 14:41:02 UTC 2022


On Thu, 24 Feb 2022 22:12:46 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> In SKARA-1063, I introduced UncheckedRestException in order to avoid logging for errors that are very likely transient, and that are tracked with metrics as well. In some cases, specifically when running Jcheck in a bot, we end up wrapping that exception in a RuntimeException, which in turn causes us to log an error instead of warning. 
> 
> This patch tries to fix this by also checking nested exceptions in when catching RuntimeException in the BotRunner.

This pull request has now been integrated.

Changeset: b97f76f4
Author:    Erik Joelsson <erikj at openjdk.org>
URL:       https://git.openjdk.java.net/skara/commit/b97f76f4621406b331950fb672ebbe15f636139a
Stats:     7 lines in 1 file changed: 6 ins; 0 del; 1 mod

Log warning on wrapped UncheckedRestException

Reviewed-by: ihse, kcr

-------------

PR: https://git.openjdk.java.net/skara/pull/1292


More information about the skara-dev mailing list