[GitHub] [iceberg] rymurr commented on pull request #2797: Upgrade to JUnit 5

[GitBox <gi...@apache.org>]

rymurr commented on pull request #2797:
URL: https://github.com/apache/iceberg/pull/2797#issuecomment-877138901


   @nastra just to double check, this doesn't change anything in terms of test order, log messages, etc it only enables us to use the new junit. Correct?
   
   What does mixed junit4 and junit5 look like? Can we have a module or class w/ annotations from both versions? Is there any danger in that? Would the recommendation be to use junit5 on new tests?
   
   cc @rdblue @aokolnychyi @RussellSpitzer 


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org