tomighita commented on PR #2120: URL: https://github.com/apache/iceberg-rust/pull/2120#issuecomment-4097919779
Thanks @blackmwk for the second round of review. I've added your suggestions and even moved the integration tests. As for the disable auto assign id part: we built a platform where we have our own schema registry. It converts data from proto to arrow and then parquet. Because of that, we want to control the assingment of ids based on how the proto definiton evolves. -- 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]
