nandorKollar commented on PR #13186: URL: https://github.com/apache/iceberg/pull/13186#issuecomment-3583329498
> Thanks for working on this feature @nandorKollar Generally LGTM, i added a few minor nits. > > Have you ran the integration test? I can look to see if i can set up the necessary infra and run the test Thanks @kevinjqliu for reviewing the PR! Yes, the new test cases which I added ran against Azure public cloud, and they passed. I'll address your comments soon. -- 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]
