szlta commented on PR #4868: URL: https://github.com/apache/iceberg/pull/4868#issuecomment-1152461200
@rdblue I can accept this reasoning. I just hope we won't have to do this change in the future either, but thanks for sharing your insights. I have amended my original change at https://github.com/apache/iceberg/pull/4662/ so that this name generation discrepancy would not cause a test failure there while reusing old partition fields.. Kindly take a look on that PR, while I'll go ahead and close this. Thanks! -- 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]
