FANNG1 commented on PR #9972: URL: https://github.com/apache/gravitino/pull/9972#issuecomment-3890593176
> 1. OK for me to upgrade the minor version. It usually only includes fixes and improvements. It won't bring > incompatible changes. > 2. It's ok to change unit tests, but I have questions. > 3. Yes, we would better to create an issue first. > > Pull request title would better follow the style `[#issue-id] improvement(iceberg): Upgrade Iceberg to 1.10.1` Could u remove unused information in your pull request description like `(Please list the user-facing changes introduced by your change, including Change in user-facing APIs. Addition or removal of property keys.)` LGTM, @agnes-xinyi-lu thanks for the PR, would you mind changing the title and description ? -- 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]
