TwinklerG commented on PR #2451:
URL: https://github.com/apache/iceberg-rust/pull/2451#issuecomment-4483746141

   > I'm not sure if this is the correct direction: it changes the 
update_schema API, and I don't think it adds much value for users/devs
   > 
   > Could you share more about your reasoning here?
   
   Well, I have been continuously tracking the Issues #2219 #697 and the 
related PRs. I noticed that the merged PR #2120 only added the capabilities of 
adding column and deleting column, but this is obviously only a small part of 
the schema evolution, and I don't think the original implementation was a good 
one. Therefore, I imitated the implementation of iceberg java and completed 
this refactoring


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to