dimas-b commented on code in PR #3839:
URL: https://github.com/apache/polaris/pull/3839#discussion_r2850088673
##########
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:
... because this change forces users of this code to adopt JUnit 6, while
traditionally Polaris required only JUnit 5. It would be nice to not rush this
change into 1.4.0, IMHO.
--
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]
