Michael Blow has submitted this change and it was merged. Change subject: [NO ISSUE][TEST] Reduce INFO verbosity on poll exception ......................................................................
[NO ISSUE][TEST] Reduce INFO verbosity on poll exception Change-Id: Id47101eeb96677ea94651c125f698a5dfffe3854 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2370 Sonar-Qube: Jenkins <[email protected]> Reviewed-by: Murtadha Hubail <[email protected]> Tested-by: Michael Blow <[email protected]> --- M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Jenkins: No violations found Michael Blow: Verified Murtadha Hubail: Looks good to me, approved diff --git a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java index eeea05a..bae1a21 100644 --- a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java +++ b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java @@ -1308,9 +1308,11 @@ e = ee; } if (e instanceof ComparisonException) { - LOGGER.log(Level.INFO, "Comparison failure on poll: " + e.getMessage()); + LOGGER.info("Comparison failure on poll: {}", e::getMessage); + } else if (LOGGER.isDebugEnabled()) { + LOGGER.debug("Received exception on poll", e); } else { - LOGGER.log(Level.INFO, "received exception on poll", e); + LOGGER.info("Received exception on poll: {}", e::toString); } responsesReceived++; if (isExpected(e, cUnit)) { -- To view, visit https://asterix-gerrit.ics.uci.edu/2370 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id47101eeb96677ea94651c125f698a5dfffe3854 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]>
