[GitHub] [cxf] reta commented on a change in pull request #849: CXF-7543: JAX-RS Features not used in proxies or WebClients

[GitBox <gi...@apache.org>]

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: dev-unsubscribe@cxf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org