AndreaBozzo commented on PR #1953: URL: https://github.com/apache/iceberg-rust/pull/1953#issuecomment-3685898805
Thanks for the detailed feedback @liurenjie1024 and @CTTY! I appreciate a lot your time and patience on this. I've addressed the review comments in commit [f09e5e7](https://github.com/apache/iceberg-rust/commit/f09e5e77c387b5ea309ffd61ab809e94a9b67307): The memory catalog remains the default for now. Full catalog loader integration (REST, Glue, HMS, etc.) would have added MANY new dependencies and i wanted to keep it simple to review/modify eventually. Thanks again. -- 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]
