diff --git a/core/src/main/java/org/opensearch/sql/expression/ip/IPFunctions.java b/core/src/main/java/org/opensearch/sql/expression/ip/IPFunctions.java index 71f9da5ab4..8b3ee23014 100644 --- a/core/src/main/java/org/opensearch/sql/expression/ip/IPFunctions.java +++ b/core/src/main/java/org/opensearch/sql/expression/ip/IPFunctions.java @@ -57,5 +57,4 @@ private ExprValue exprCidrMatch(ExprValue addressExprValue, ExprValue rangeExprV ? ExprValueUtils.LITERAL_FALSE : ExprValueUtils.LITERAL_TRUE; } - } diff --git a/core/src/test/java/org/opensearch/sql/expression/ip/GeoIPFunctionTest.java b/core/src/test/java/org/opensearch/sql/expression/ip/GeoIPFunctionTest.java index 8f5136d710..dcda03d5a6 100644 --- a/core/src/test/java/org/opensearch/sql/expression/ip/GeoIPFunctionTest.java +++ b/core/src/test/java/org/opensearch/sql/expression/ip/GeoIPFunctionTest.java @@ -18,7 +18,6 @@ import org.opensearch.sql.data.model.ExprValue; import org.opensearch.sql.expression.DSL; import org.opensearch.sql.expression.Expression; -import org.opensearch.sql.expression.FunctionExpression; import org.opensearch.sql.expression.env.Environment; @ExtendWith(MockitoExtension.class) diff --git a/opensearch/src/main/java/org/opensearch/sql/opensearch/executor/protector/OpenSearchExecutionProtector.java b/opensearch/src/main/java/org/opensearch/sql/opensearch/executor/protector/OpenSearchExecutionProtector.java index e4423b5df9..235ddc0075 100644 --- a/opensearch/src/main/java/org/opensearch/sql/opensearch/executor/protector/OpenSearchExecutionProtector.java +++ b/opensearch/src/main/java/org/opensearch/sql/opensearch/executor/protector/OpenSearchExecutionProtector.java @@ -99,10 +99,10 @@ public PhysicalPlan visitRemove(RemoveOperator node, Object context) { public PhysicalPlan visitEval(EvalOperator node, Object context) { if (node instanceof OpenSearchEvalOperator evalOperator) { return doProtect( - new OpenSearchEvalOperator( - visitInput(evalOperator.getInput(), context), - evalOperator.getExpressionList(), - evalOperator.getNodeClient())); + new OpenSearchEvalOperator( + visitInput(evalOperator.getInput(), context), + evalOperator.getExpressionList(), + evalOperator.getNodeClient())); } return new EvalOperator(visitInput(node.getInput(), context), node.getExpressionList()); } diff --git a/opensearch/src/test/java/org/opensearch/sql/opensearch/executor/protector/OpenSearchExecutionProtectorTest.java b/opensearch/src/test/java/org/opensearch/sql/opensearch/executor/protector/OpenSearchExecutionProtectorTest.java index 874777e21b..b2a151d2e6 100644 --- a/opensearch/src/test/java/org/opensearch/sql/opensearch/executor/protector/OpenSearchExecutionProtectorTest.java +++ b/opensearch/src/test/java/org/opensearch/sql/opensearch/executor/protector/OpenSearchExecutionProtectorTest.java @@ -346,21 +346,20 @@ public void test_visitTrendline() { void test_visitOpenSearchEval() { NodeClient nodeClient = mock(NodeClient.class); OpenSearchEvalOperator evalOperator = -// ADOperator adOperator = - new OpenSearchEvalOperator( - values(emptyList()), - List.of( - ImmutablePair.of( - new ReferenceExpression("ageInAbs", OpenSearchTextType.of()), - DSL.abs(DSL.abs(new ReferenceExpression("age", ExprCoreType.LONG)))) - ), - nodeClient); + // ADOperator adOperator = + new OpenSearchEvalOperator( + values(emptyList()), + List.of( + ImmutablePair.of( + new ReferenceExpression("ageInAbs", OpenSearchTextType.of()), + DSL.abs(DSL.abs(new ReferenceExpression("age", ExprCoreType.LONG))))), + nodeClient); assertEquals( - executionProtector.doProtect(evalOperator), executionProtector.visitEval(evalOperator, null)); + executionProtector.doProtect(evalOperator), + executionProtector.visitEval(evalOperator, null)); } - PhysicalPlan resourceMonitor(PhysicalPlan input) { return new ResourceMonitorPlan(input, resourceMonitor); }