Skip to content

Commit

Permalink
[DM-37254] Emit exception message as warning
Browse files Browse the repository at this point in the history
Right now this is printed out at the debug level, but it will be a
lot easier to find and get a hold of if we make it a warning.  Since
this is an exception path, this shouldn't really add too much log
spam, considering the rethrown messages are emitted after, but we
seem to lose the first one which is the most important.
  • Loading branch information
cbanek committed Dec 20, 2022
1 parent 56df28d commit 10ab755
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/main/java/org/opencadc/tap/impl/QServQueryRunner.java
Original file line number Diff line number Diff line change
Expand Up @@ -496,8 +496,8 @@ else if (rs != null)
diagnostics.add(new Result("diag", URI.create("fail:"+dt)));

errorMessage = t.getClass().getSimpleName() + ":" + t.getMessage();
log.debug("BADNESS", t);
log.debug("Error message: " + errorMessage);
log.warn("BADNESS", t);
log.warn("Error message: " + errorMessage);

log.debug("creating TableWriter for error...");
TableWriter ewriter = pfac.getErrorWriter();
Expand Down

0 comments on commit 10ab755

Please sign in to comment.