nastra commented on pull request #4333: URL: https://github.com/apache/iceberg/pull/4333#issuecomment-1076115697
> If we add this to `core`, what are the implications for existing JUnit 4 based tests? As well as for things like overall runtime or any other concerns in operating in a mixed mode? Possibly classpath issues? > > Trying to better understand the implications of adding it. Existing Junit4 tests will work as usual, so there are no implications and no classpath issues. The only implication is that subclasses of `CatalogTests` will have to be Junit5 tests (which is what's being addressed here as well) -- 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]
