cjj2010 commented on code in PR #4818:
URL: https://github.com/apache/calcite/pull/4818#discussion_r2909019488


##########
core/src/main/java/org/apache/calcite/sql/SqlKind.java:
##########
@@ -1357,7 +1357,10 @@ public enum SqlKind {
    * commands for them. Use OTHER_DDL in the short term, but we are happy to 
add
    * new enum values for your object types. Just ask!
    */
-  OTHER_DDL;
+  OTHER_DDL,
+
+  /** The {@code REGEXP} function. */
+  REGEXP;

Review Comment:
   > Why make this change? If it were a dialect conversion, it could be 
determined using `SqlOperator`.
   
   Can we modify the function to
   SqlBasicFunction.create(SqlKind.RLIKE, ReturnTypes.BOOLEAN_NULLABLE,
   OperandTypes.STRING_STRING);
   Using SQL Kind.RLIKE, it seems that there is a need for a kind in HiveSQL 
Dialect to perform conversion judgments, and I am not sure if I understand it 
correctly



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to