jackye1995 opened a new issue #2724:
URL: https://github.com/apache/iceberg/issues/2724


   continuing the discussion in #2560 
   
   Currently I introduced the following syntax for identifier fields:
   
   ```sql
   ALTER TABLE table SET IDENTIFIER FIELDS col1, col2, col3
   ALTER TABLE table DROP IDENTIFIER FIELDS col1,col2
   ```
   
   In the end I had a thought that the above syntax is trying to mimic the Hive 
standard, but we are not really using that standard in Spark extensions anyway. 
It might be better to make it consistent with sort order update syntax, with 
something like:
   
   ```sql
   ALTER TABLE table WRITE IDENTIFIED BY col1,col2;
   ```
   
   And we do not support add/drop to avoid confusion.
   
   Any thoughts? @rdblue @aokolnychyi @openinx 


-- 
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to