From 53ed2136cd0514e13c967250b4c727324a7ea637 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Grzegorz=20Kokosi=C5=84ski?= Date: Tue, 29 Jan 2019 08:38:01 +0100 Subject: [PATCH] Log test query durarion only if it run longer than 1s --- .../main/java/io/prestosql/tests/QueryAssertions.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/presto-tests/src/main/java/io/prestosql/tests/QueryAssertions.java b/presto-tests/src/main/java/io/prestosql/tests/QueryAssertions.java index 459973d45b70..08080a1867a1 100644 --- a/presto-tests/src/main/java/io/prestosql/tests/QueryAssertions.java +++ b/presto-tests/src/main/java/io/prestosql/tests/QueryAssertions.java @@ -70,7 +70,10 @@ public static void assertUpdate(QueryRunner queryRunner, Session session, @Langu queryPlan = null; results = queryRunner.execute(session, sql); } - log.info("FINISHED in presto: %s", nanosSince(start)); + Duration queryTime = nanosSince(start); + if (queryTime.compareTo(Duration.succinctDuration(1, SECONDS)) > 0) { + log.info("FINISHED in presto: %s", queryTime); + } if (planAssertion.isPresent()) { planAssertion.get().accept(queryPlan); @@ -160,7 +163,10 @@ private static void assertQuery( catch (RuntimeException ex) { fail("Execution of 'expected' query failed: " + expected, ex); } - log.info("FINISHED in presto: %s, h2: %s, total: %s", actualTime, nanosSince(expectedStart), nanosSince(start)); + Duration totalTime = nanosSince(start); + if (totalTime.compareTo(Duration.succinctDuration(1, SECONDS)) > 0) { + log.info("FINISHED in presto: %s, h2: %s, total: %s", actualTime, nanosSince(expectedStart), totalTime); + } if (actualResults.getUpdateType().isPresent() || actualResults.getUpdateCount().isPresent()) { if (!actualResults.getUpdateType().isPresent()) {