amogh-jahagirdar commented on PR #16077: URL: https://github.com/apache/iceberg/pull/16077#issuecomment-4355466234
Thanks @hemanthboyina! and @huaxingao @RussellSpitzer for reviewing. +1 to Russell's point about using the constants in a follow up. I'll go ahead and merge but if someone has the cycles to move to using constants in a cycle that'd be great to cleanup. -- 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]
