dimas-b commented on code in PR #2782:
URL: https://github.com/apache/polaris/pull/2782#discussion_r2422182379
##########
runtime/service/src/main/java/org/apache/polaris/service/credentials/connection/SigV4ConnectionCredentialVendor.java:
##########
@@ -58,7 +58,7 @@
* <p>This is the default implementation with {@code @Priority(100)}. Custom
implementations can
* override this by providing a higher priority value.
*/
-@ApplicationScoped
+@RequestScoped
Review Comment:
`DefaultServiceIdentityRegistry` does not exist in code? :thinking: Did you
mean `DefaultServiceIdentityProvider`?
Referencing an application-scoped object from a request scope is usually ok
:thinking:
--
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]
