joewee commented on code in PR #4326:
URL: https://github.com/apache/calcite/pull/4326#discussion_r2055394264


##########
core/src/main/java/org/apache/calcite/sql/SqlDelete.java:
##########
@@ -32,7 +32,20 @@
  */
 public class SqlDelete extends SqlCall {
   public static final SqlSpecialOperator OPERATOR =
-      new SqlSpecialOperator("DELETE", SqlKind.DELETE);
+      new SqlSpecialOperator("DELETE", SqlKind.DELETE) {
+        @SuppressWarnings("argument.type.incompatible")
+        @Override public SqlCall createCall(
+            @Nullable SqlLiteral functionQualifier,
+            SqlParserPos pos,
+            @Nullable SqlNode... operands) {
+          return new SqlDelete(
+              pos,
+              operands[0],
+              operands[1],
+              operands.length > 2 ? (SqlSelect) operands[2] : null,

Review Comment:
   To be exact, we don't know where the operands are taken from. The only thing 
we can be sure of is that there are exactly 4 parameters when getting them from 
the getOperandList method. In other cases, we cannot be sure of the number of 
operands. @xiedeyantu 



-- 
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