nastra commented on PR #9793: URL: https://github.com/apache/iceberg/pull/9793#issuecomment-2034605250
> @nastra yes, I'm gonna made changes you proposed. In majority changes I switched Junit4 to Junit5 not AssertJ because it was said to switch to Junit5. Anyway, thank you for your comments I'm going to make changes and update PR asap. Thanks, that sounds great. Also see https://iceberg.apache.org/contribute/#assertj -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org