[GitHub] [iceberg] yyanyy commented on issue #2724: Spark SQL extension syntax for identifier field update

[GitBox <gi...@apache.org>]

yyanyy commented on issue #2724:
URL: https://github.com/apache/iceberg/issues/2724#issuecomment-867177322


   Sorry I didn't follow the rest of the discussion in the PR/it's been a while since I looked last time, I thought UNSET/DROP removes all identifier fields from the table. Now it seems that we can selectively drop some fields but not the others. Is this the case? I'm not sure if I like the idea of dropping some but not all fields since the user has to know what the original list was. 
   
   I think `WRITE IDENTIFIED BY col1,col2;` looks good for setting and replacing identifier fields, but I wonder what the command should be if we want to drop everything. Do we plan to make it `ALTER TABLE table WRITE IDENTIFIED BY;`? Would this look a bit odd as if the command hasn't finished, and potentially error prone if someone accidentally press `;` and enter when they are still coming up with the list? `UNSET`/`DROP` requires a completely different keyword which may reduce this risk. 


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



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