liurenjie1024 commented on PR #349: URL: https://github.com/apache/iceberg-rust/pull/349#issuecomment-2510452060
> @ZENOTME thanks for getting back to me here. I don't want to be the one holding this up, so I would suggest that we get this in. There are still some gaps I see because I think it is pretty easy to brick a table by adding some incompatible files. I'll raise new issues for that so we can pick up those independently. > > Any concerns @c-thiel @liurenjie1024 @Xuanwo ? Hi, @Fokko sorry for late reply, I took a pr and agree that there are still some improvements, and we can resolve them in follow up issues. -- 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-unsubscr...@iceberg.apache.org 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
