Skip to content

Commit 1ab0296

Browse files
committed
remove duplicated regex match operator alias
Signed-off-by: Jialiang Liang <jiallian@amazon.com>
1 parent 9f52003 commit 1ab0296

3 files changed

Lines changed: 1 addition & 8 deletions

File tree

core/src/main/java/org/opensearch/sql/calcite/CalciteRelNodeVisitor.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -183,9 +183,7 @@ public RelNode visitRegex(Regex node, CalcitePlanContext context) {
183183

184184
RexNode regexCondition =
185185
context.rexBuilder.makeCall(
186-
org.opensearch.sql.expression.function.PPLBuiltinOperators.REGEX_MATCH,
187-
fieldRex,
188-
patternRex);
186+
org.apache.calcite.sql.fun.SqlLibraryOperators.REGEXP_CONTAINS, fieldRex, patternRex);
189187

190188
if (node.isNegated()) {
191189
regexCondition = context.rexBuilder.makeCall(SqlStdOperatorTable.NOT, regexCondition);

core/src/main/java/org/opensearch/sql/expression/function/PPLBuiltinOperators.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
import org.apache.calcite.linq4j.tree.Expression;
2323
import org.apache.calcite.rex.RexCall;
2424
import org.apache.calcite.sql.SqlOperator;
25-
import org.apache.calcite.sql.fun.SqlLibraryOperators;
2625
import org.apache.calcite.sql.type.ReturnTypes;
2726
import org.apache.calcite.sql.type.SqlTypeTransforms;
2827
import org.apache.calcite.sql.util.ReflectiveSqlOperatorTable;
@@ -394,8 +393,6 @@ public class PPLBuiltinOperators extends ReflectiveSqlOperatorTable {
394393
new org.opensearch.sql.expression.function.udf.binning.RangeBucketFunction()
395394
.toUDF("RANGE_BUCKET");
396395

397-
public static final SqlOperator REGEX_MATCH = SqlLibraryOperators.REGEXP_CONTAINS;
398-
399396
/**
400397
* Returns the PPL specific operator table, creating it if necessary.
401398
*

core/src/main/java/org/opensearch/sql/expression/function/PPLFuncImpTable.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,6 @@
160160
import static org.opensearch.sql.expression.function.BuiltinFunctionName.RAND;
161161
import static org.opensearch.sql.expression.function.BuiltinFunctionName.REDUCE;
162162
import static org.opensearch.sql.expression.function.BuiltinFunctionName.REGEXP;
163-
import static org.opensearch.sql.expression.function.BuiltinFunctionName.REGEX_MATCH;
164163
import static org.opensearch.sql.expression.function.BuiltinFunctionName.REPLACE;
165164
import static org.opensearch.sql.expression.function.BuiltinFunctionName.REVERSE;
166165
import static org.opensearch.sql.expression.function.BuiltinFunctionName.RIGHT;
@@ -713,7 +712,6 @@ void populate() {
713712
registerOperator(SIMPLE_QUERY_STRING, PPLBuiltinOperators.SIMPLE_QUERY_STRING);
714713
registerOperator(QUERY_STRING, PPLBuiltinOperators.QUERY_STRING);
715714
registerOperator(MULTI_MATCH, PPLBuiltinOperators.MULTI_MATCH);
716-
registerOperator(REGEX_MATCH, PPLBuiltinOperators.REGEX_MATCH);
717715

718716
// Register PPL Datetime UDF operator
719717
registerOperator(TIMESTAMP, PPLBuiltinOperators.TIMESTAMP);

0 commit comments

Comments
 (0)