dimas-b commented on code in PR #3839:
URL: https://github.com/apache/polaris/pull/3839#discussion_r2829911568
##########
integration-tests/src/main/java/org/apache/polaris/service/it/ext/PolarisIntegrationTestExtension.java:
##########
@@ -68,7 +68,7 @@ public Object resolveParameter(
private Env env(ExtensionContext context) {
ExtensionContext classCtx = classContext(context);
ExtensionContext.Store store = classCtx.getStore(NAMESPACE);
- return store.getOrComputeIfAbsent(
+ return store.computeIfAbsent(
Review Comment:
This will force all users of this extension to adopt JUnit 6. I do not have
any data on whether this will have a significant ripple effect or not, but it
might be best to do this after 1.4.0
--
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]