From a4c7745f15b21c8b0d86bf2e878b38da021720c8 Mon Sep 17 00:00:00 2001 From: Raunaq Morarka Date: Thu, 24 Feb 2022 17:19:45 +0530 Subject: [PATCH] Remove redundant firstNonNaN method --- .../src/main/java/io/trino/cost/JoinStatsRule.java | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/core/trino-main/src/main/java/io/trino/cost/JoinStatsRule.java b/core/trino-main/src/main/java/io/trino/cost/JoinStatsRule.java index 79bb2eded80f..04ae4b6018ce 100644 --- a/core/trino-main/src/main/java/io/trino/cost/JoinStatsRule.java +++ b/core/trino-main/src/main/java/io/trino/cost/JoinStatsRule.java @@ -39,8 +39,8 @@ import static io.trino.sql.ExpressionUtils.extractConjuncts; import static io.trino.sql.planner.plan.Patterns.join; import static io.trino.sql.tree.ComparisonExpression.Operator.EQUAL; +import static io.trino.util.MoreMath.firstNonNaN; import static java.lang.Double.NaN; -import static java.lang.Double.isNaN; import static java.lang.Math.min; import static java.util.Comparator.comparingDouble; import static java.util.Objects.requireNonNull; @@ -250,16 +250,6 @@ private PlanNodeStatsEstimate filterByAuxiliaryClause(PlanNodeStatsEstimate stat return normalizer.normalize(result.build(), types); } - private static double firstNonNaN(double... values) - { - for (double value : values) { - if (!isNaN(value)) { - return value; - } - } - throw new IllegalArgumentException("All values are NaN"); - } - /** * Calculates statistics for unmatched left rows. */