[GitHub] johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0

2017-12-18 Thread GitBox
johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0 URL: https://github.com/apache/cxf/pull/359#discussion_r157662913 ## File path:

[GitHub] johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0

2017-12-18 Thread GitBox
johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0 URL: https://github.com/apache/cxf/pull/359#discussion_r157662779 ## File path:

[GitHub] johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0

2017-12-18 Thread GitBox
johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0 URL: https://github.com/apache/cxf/pull/359#discussion_r157662654 ## File path:

[GitHub] johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0

2017-12-18 Thread GitBox
johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0 URL: https://github.com/apache/cxf/pull/359#discussion_r157662550 ## File path:

[GitHub] johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0

2017-12-18 Thread GitBox
johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0 URL: https://github.com/apache/cxf/pull/359#discussion_r157662397 ## File path: rt/rs/microprofile-client/pom.xml ## @@ -0,0 +1,230 @@ + +

[GitHub] johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0

2017-12-18 Thread GitBox
johnament commented on a change in pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0 URL: https://github.com/apache/cxf/pull/359#discussion_r157662423 ## File path: rt/rs/microprofile-client/pom.xml ## @@ -0,0 +1,230 @@ + +

[GitHub] johnament opened a new pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0

2017-12-18 Thread GitBox
johnament opened a new pull request #359: Adding an implementation of the MicroProfile Rest Client v1.0 URL: https://github.com/apache/cxf/pull/359 This is an implementation of the MP Client 1.0. I plan to cut the spec release tomorrow, so don't quite approve this yet but need input on

[GitHub] reta commented on a change in pull request #351: [CXF-7571] Adding support for CDI injection of @Context objects.

2017-12-18 Thread GitBox
reta commented on a change in pull request #351: [CXF-7571] Adding support for CDI injection of @Context objects. URL: https://github.com/apache/cxf/pull/351#discussion_r157485092 ## File path: integration/cdi/src/main/java/org/apache/cxf/cdi/DelegateContextAnnotatedType.java

Re: Default Priority for built in providers

2017-12-18 Thread Sergey Beryozkin
At some point, user-registered providers were always preferred over the equal built-in providers before even starting the selection,, but eventually, after the TCK failures and long discussions, it proved CXF was not compliant at a time. Thus, first, the built in and user-registered providers

Re: Default Priority for built in providers

2017-12-18 Thread Sergey Beryozkin
What exactly is falling down ? Sergey On 17/12/17 21:46, John D. Ament wrote: I figured out where its falling down. If you look at the class ProviderInfoClassComparator in ProviderFactory, it handles sort the provider infos with the custom attribute set. The problem is that the actual

Re: Default Priority for built in providers

2017-12-18 Thread Sergey Beryozkin
Which default providers are you referring to ? If it is MBR or MBW then their priority is implicit, based on the spec text re how to sort media types, etc. Sergey On 17/12/17 14:42, John D. Ament wrote: FWIW, I had assumed I was doing something wrong. However, I'm just delegating down to