From c38757e7e4cd0bfabe5b428d65568cbdc18286e8 Mon Sep 17 00:00:00 2001 From: Neha Harish Date: Wed, 8 Dec 2021 10:48:06 +0530 Subject: [PATCH] return value ignored no side effect fix --- .../prestosql/plugin/splitmanager/TestStepCalcManage.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/presto-base-jdbc/src/test/java/io/prestosql/plugin/splitmanager/TestStepCalcManage.java b/presto-base-jdbc/src/test/java/io/prestosql/plugin/splitmanager/TestStepCalcManage.java index 797fadbc8..d81ddb011 100644 --- a/presto-base-jdbc/src/test/java/io/prestosql/plugin/splitmanager/TestStepCalcManage.java +++ b/presto-base-jdbc/src/test/java/io/prestosql/plugin/splitmanager/TestStepCalcManage.java @@ -15,6 +15,7 @@ package io.prestosql.plugin.splitmanager; import com.google.common.collect.ImmutableList; +import io.airlift.log.Logger; import io.airlift.units.Duration; import org.testng.annotations.Test; @@ -27,6 +28,8 @@ import static org.testng.Assert.assertTrue; public class TestStepCalcManage { + private static final Logger log = Logger.get(TestStepCalcManage.class); + @Test // all splits are equally distributed, no adjust public void testDynamicStepCalc0() @@ -677,7 +680,7 @@ public class TestStepCalcManage Thread.sleep(10); } catch (InterruptedException e) { - e.getMessage(); + log.debug("Interrupted Exception %s", e.getMessage()); } result = stepCalcManager.getAdjustSplitList(tableConfig.getCatalogName(), tableConfig.getSchemaName(), tableConfig.getTableName()); assertEquals(result, null); @@ -708,7 +711,7 @@ public class TestStepCalcManage Thread.sleep(10); } catch (InterruptedException e) { - e.getMessage(); + log.debug("Interrupted Exception %s", e.getMessage()); } result = stepCalcManager.getAdjustSplitList(tableConfig.getCatalogName(), tableConfig.getSchemaName(), tableConfig.getTableName()); assertEquals(result, null);