reta commented on a change in pull request #849:
URL: https://github.com/apache/cxf/pull/849#discussion_r706946473
##########
File path:
rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/JAXRSClientFactoryBean.java
##########
@@ -403,13 +412,45 @@ protected void initClient(AbstractClient client, Endpoint
ep, boolean addHeaders
});
}
}
+
+ protected <C extends Configurable<C>> Configurable<?> getConfigurableFor(C
context) {
Review comment:
Thanks a lot for looking, @andymc12 ! I was also initially though of
"create" naming but in case of MP, it turned out to be not exactly the case:
the existing instance of the configurable is used for any context. In this
regards, `getConfigurableFor` is a bit more honest name with respect to
semantics, but `getOrCreateConfigurableFor` would also make sense. What do you
think?
--
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]
