LadyForest commented on code in PR #21571:
URL: https://github.com/apache/flink/pull/21571#discussion_r1059269728
##########
flink-table/flink-sql-parser/src/main/java/org/apache/flink/sql/parser/ddl/SqlAlterTableDropConstraint.java:
##########
@@ -24,25 +24,29 @@
import org.apache.calcite.sql.parser.SqlParserPos;
import org.apache.calcite.util.ImmutableNullableList;
+import javax.annotation.Nullable;
+
import java.util.List;
/** ALTER TABLE [catalog_name.][db_name.]table_name DROP CONSTRAINT
constraint_name. */
public class SqlAlterTableDropConstraint extends SqlAlterTable {
+
private final SqlIdentifier constraintName;
/**
* Creates an alter table drop constraint node.
*
- * @param tableID Table ID
- * @param constraintName Constraint name
* @param pos Parser position
+ * @param tableName Table name
+ * @param constraintName Constraint name
*/
public SqlAlterTableDropConstraint(
- SqlIdentifier tableID, SqlIdentifier constraintName, SqlParserPos
pos) {
- super(pos, tableID);
+ SqlParserPos pos, SqlIdentifier tableName, SqlIdentifier
constraintName) {
+ super(pos, tableName);
this.constraintName = constraintName;
}
+ @Nullable
Review Comment:
This should be removed.
--
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]
