jackye1995 opened a new pull request #2560: URL: https://github.com/apache/iceberg/pull/2560
continuation of #2556 , will need to rebase after that is merged. Here I propose the following syntax: ```sql ALTER TABLE table REPLACE IDENTIFIER FIELDS (field1, field2, ...) -- drop all ALTER TABLE table DROP IDENTIFIER FIELDS ``` `IDENTIFIER` and `FIELDS` are new keywords. Because we already have `FIELD` as a keyword, I think adding `FIELDS` can be beneficial for other batch operations in the future. It also makes the statement more clear instead of saying `ALTER TABLE table REPLACE IDENTIFIER`. The reason for having another drop statement to drop all is because typically in SQL statements a `( ... )` clause always has at least one element. We can do `ALTER TABLE table REPLACE IDENTIFIER FIELDS ()` instead, but it just feels awkward to me, not sure how other people think about this. -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
