[GitHub] [iceberg] jackye1995 commented on pull request #2818: Core: Convert Type for update schema

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

jackye1995 commented on pull request #2818:
URL: https://github.com/apache/iceberg/pull/2818#issuecomment-881642743


   I think the reason Iceberg only does the 3 conversions now is because of partition spec compatibility. For example, if a column `id` is of int type, and has a partition field `bucket(16, id)`, and you change it to float type, then the partition transformation would fail.
   
   I think it is a very great idea to have a matrix for compatibility, but we need to check one by one to see if any partition transform would break, and have a matrix for Iceberg.
   
   In addition, allowing too many update options would limit our ability to introduce new partition transform functions. I don't see any new ones added soon, but I don't know if anyone has some thinking about some other potentially beneficial transforms to have in long run that we should discuss. Probably worth brainstorming in the dev list.


-- 
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: issues-unsubscribe@iceberg.apache.org

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