dimas-b commented on code in PR #2544:
URL: https://github.com/apache/polaris/pull/2544#discussion_r2338065771
##########
polaris-core/src/test/java/org/apache/polaris/service/storage/gcp/GcpCredentialsStorageIntegrationTest.java:
##########
@@ -177,6 +177,12 @@ private AccessConfig subscopedCredsForOperations(
Optional.of(REFRESH_ENDPOINT));
}
+ private JsonNode readResource(ObjectMapper mapper, String name) throws
IOException {
+ // readTree from a test resource is not affected by the issues that
prompted its deprecation
+ //noinspection deprecation
+ return
mapper.readTree(GcpCredentialsStorageIntegrationTest.class.getResource(name));
Review Comment:
I thought it might be simpler to suppress for test code... but it's a good
point. I'll update.
--
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]
