Re: CDI integration
Sergey Beryozkin
Sorry for the typo, I meant to say
that "ignoring of the fact that major non-EE containers are *now*
thought by"
On 31/05/17 12:19, Sergey Beryozkin wrote:
|
|
Re: CDI integration
Sergey Beryozkin
On 31/05/17 12:09, Arjan Tijms wrote:
On Wed, May 31, 2017 at 04:00 am, Sergey Beryozkin wrote:Because this is what non EE users of JAX-RS (and there are many of them) look for (they migrate to Spring Boot and non-EE containers in general). I've always felt that this ignoring of the fact that major non-EE containers are not thought by many many users is not good, as far as restricting a given (JAX-RS) API is concerned Cheers, Sergey
|
|
Re: CDI integration
Sergey Beryozkin
On 31/05/17 12:07, Arjan Tijms wrote:
Technically it is. Not all nderstand though why JAX-RS is one of the most popular EE spec today - let me answer, because the developers could use it everywhere without having to ensure some EE Managed ExecutorService exists around. Thanks, Sergey
|
|
Re: CDI integration
On Wed, May 31, 2017 at 04:00 am, Sergey Beryozkin wrote:
JAX-RS is big in EE but it needs to be present in SpringBoot, etc. Why?
|
|
Re: CDI integration
On Wed, May 31, 2017 at 12:53 PM, Sergey Beryozkin <sberyozkin@...> wrote:
Yes. It's another injection and bean model and doesn't play nice with the platform wide services for things such as extensions, decorators and interceptors. This is also why we deprecated and native managed beans in JSF. Java EE is not as good as it could be now, because many specs live on their own little islands.
JAX-RS is a part of Java EE, is it not? Kind regards, Arjan Tijms
|
|
Re: CDI integration
Sergey Beryozkin
Let me step back. +1 to having a better CDI integration, and in CXF my colleagues work hard on making it better. @Context needs to be continued to be supported though. JAX-RS is big in EE but it needs to be present in SpringBoot, etc. Wiring CDI into non EE can become problematic thus continuing to have a basic @Context support is good. Sorry for hajacking the thread Cheers, Sergey
On 31/05/17 11:53, Sergey Beryozkin wrote:
|
|
Re: CDI integration
Sergey Beryozkin
Hi Lest not dramatize it, difficult, confusing, all because of @Context ? I'm not going to comment. As well as re this 'ownership' claim of JAX-RS by EE. Sergey
On 31/05/17 11:44, Arjan Tijms wrote:
|
|
Re: CDI integration
Hi, On Wed, May 31, 2017 at 11:52 AM, Sergey Beryozkin <sberyozkin@...> wrote:
It's not just another EE spec, but one of the fundamental EE specs.
IMHO; open as possible also means that you eventually commit to nothing and only make it more difficult and confusing to use in its primary environment. Kind regards. Arjan Tijms
|
|
Re: CDI integration
Sergey Beryozkin
@Context is that little thing that has
helped to make JAX-RS visible outside of the EE world, as opposed
to it be yet another EE spec. Rather than trying to keep JAX-RS more and more dependent on EE pieces I'd like to encourage us keeping it as open as possible Sergey
On 31/05/17 10:44, Sebastian Daschner wrote:
|
|
Re: CDI integration
Sebastian Daschner
Yes, agreed. Especially the ability to @Inject JAX-RS managed objects (basically what is @Context today) would be very helpful. That enables to inject UriInfo within the EE application and also getting rid of @Context to streamline Java EE further :-) Cheers,
On 05/31/2017 03:12 PM, Christian
Kaltepoth wrote:
|
|
Re: CDI integration
I agree with Guillermo and Arjan that further CDI alignment is very important for JAX-RS. It would be great to see some EG discussion about this. Christian 2017-05-25 20:13 GMT+02:00 Arjan Tijms <arjan.tijms@...>:
--
|
|
Re: PATCH support on the client
Andy McCright
That is more-or-less my opinion too. I'd prefer that we supported the patch() method fully (not optionally), but if the decision is support it optionally or not at all, I'd choose not at all.
On this topic, is it possible to remove the HttpUrlConnection restriction in Java 9 so that we could have more flexibility for future JAX-RS Client APIs?
Thanks,
Andy
J. Andrew McCright IBM WebSphere Development +1 507 253 7448 TL 553-7448 andymc@...
----- Original message -----
|
|
Re: PATCH support on the client
Alessio Soldano
Il 30/05/2017 10:48, Sergey Beryozkin ha scritto:
Re patch() - I've no strong opinion whether this actual method stays or not, because we still have method(), and that was my point when I replied to Markus,Same feeling here. Cheers Alessio
|
|
Re: PATCH support on the client
Sergey Beryozkin
Hi Pavel
toggle quoted messageShow quoted text
Right, that what I was also implying, that API already allows to use any HTTP method names, so it is not only about PATCH. So we should have method(String) variations docs updated. Re patch() - I've no strong opinion whether this actual method stays or not, because we still have method(), and that was my point when I replied to Markus, we can/should not enforce an 'either works for all the platform or not at all' solution. Having patch() staying won't change anything except for having this optional method be more visible which is what I guess adding this method was mainly about. If dropping it will make API less controversial then it is fine... Thanks, Sergey
On 30/05/17 09:13, Pavel Bucek wrote:
The issue with Sync/Async/RxInvoker#patch methods is similar to what we do have with Sync/Async/RxInvoker#method, but surprisingly #method doesn't mention anything about unsupported HTTP methods or runtime requirements. On the other hand, there seem to be general agreement about the existence of unsupported HTTP methods.
|
|
Re: PATCH support on the client
Pavel Bucek
The issue with Sync/Async/RxInvoker#patch methods is similar to what we do have with Sync/Async/RxInvoker#method, but surprisingly #method doesn't mention anything about unsupported HTTP methods or runtime requirements. On the other hand, there seem to be general agreement about the existence of unsupported HTTP methods.
toggle quoted messageShow quoted text
The problem with #patch is that it is extracted (similarly as #get, #post, ..), so anyone would assume that it has to work. If there is a strong opposition to marking those methods as optional, we can remove them completely, as suggested. #method is always there, so the functionality is not gone and it might make more sense from certain point of view. Seems like Markus already stated, that he strongly prefers to remove #patch (compared to keeping them as optional), Sergey might be ok with #patch staying int the client API (feel free to correct me). Any other opinions? (note that removing methods from the API is simple; we certainly can choose to re-add them later. We also could only lift the "optionality" and creating less disturbing change in the future). Thanks, Pavel
On 29/05/2017 18:28, Sergey Beryozkin wrote:
The issue applies not only to PATCH but to all other methods which happen not be known to HttpUrlConnection.
|
|
Re: PATCH support on the client
In fact no ISV is using that optional methods unless they package their app with a particular JRE.
toggle quoted messageShow quoted text
-Markus
-----Original Message-----
From: jaxrs-spec@javaee.groups.io [mailto:jaxrs-spec@javaee.groups.io] On Behalf Of Sergey Beryozkin Sent: Montag, 29. Mai 2017 18:29 To: jaxrs-spec@javaee.groups.io Subject: Re: [jaxrs] PATCH support on the client The issue applies not only to PATCH but to all other methods which happen not be known to HttpUrlConnection. But rather than say something along the lines "only well-known verbs" can be supported and be forever restricted by this strange HttpUrlConnection restriction given that all stacks can work with HttpClient, I'd rather have JAX-RS 2.1 be much more open. Even Collection API in java.util. has a number of optional methods. Some ISVs might decide not to use this optional method and use their own code - but that won't make their code any more portable as it will be tied to HttpClient or similar.... Sergey On 29/05/17 17:19, Markus KARG wrote: My opinion is that either we support it completely or not at all. Making it optional renders the feature useless for ISVs: If I cannot rely on it being there on _all_ platforms, I will _never_ use it, or I will _always_ bring it with me on my own. So it would be good for nothing.
|
|
Re: PATCH support on the client
Sergey Beryozkin
The issue applies not only to PATCH but to all other methods which happen not be known to HttpUrlConnection.
toggle quoted messageShow quoted text
But rather than say something along the lines "only well-known verbs" can be supported and be forever restricted by this strange HttpUrlConnection restriction given that all stacks can work with HttpClient, I'd rather have JAX-RS 2.1 be much more open. Even Collection API in java.util. has a number of optional methods. Some ISVs might decide not to use this optional method and use their own code - but that won't make their code any more portable as it will be tied to HttpClient or similar.... Sergey
On 29/05/17 17:19, Markus KARG wrote:
My opinion is that either we support it completely or not at all. Making it optional renders the feature useless for ISVs: If I cannot rely on it being there on _all_ platforms, I will _never_ use it, or I will _always_ bring it with me on my own. So it would be good for nothing.
|
|
Re: PATCH support on the client
My opinion is that either we support it completely or not at all. Making it optional renders the feature useless for ISVs: If I cannot rely on it being there on _all_ platforms, I will _never_ use it, or I will _always_ bring it with me on my own. So it would be good for nothing.
toggle quoted messageShow quoted text
-Markus
-----Original Message-----
From: jaxrs-spec@javaee.groups.io [mailto:jaxrs-spec@javaee.groups.io] On Behalf Of Pavel Bucek Sent: Montag, 29. Mai 2017 17:23 To: jaxrs-spec@javaee.groups.io Subject: [jaxrs] PATCH support on the client Dear experts, we'd like to make the HTTP PATCH support on the client optional, throwing UnsupportedOperationException on client runtimes, which don't support that method. I know that we already had a discussion about this, but we concluded that requiring something without having support from the target JDK might be too cumbersome for some environments. This wouldn't change anything on the server side, it's purely about the client (reflecting the issues related to HttpUrlConnection and derived implementations). Any comments/suggestions welcomed. Thanks, Pavel
|
|
PATCH support on the client
Pavel Bucek
Dear experts,
we'd like to make the HTTP PATCH support on the client optional, throwing UnsupportedOperationException on client runtimes, which don't support that method. I know that we already had a discussion about this, but we concluded that requiring something without having support from the target JDK might be too cumbersome for some environments. This wouldn't change anything on the server side, it's purely about the client (reflecting the issues related to HttpUrlConnection and derived implementations). Any comments/suggestions welcomed. Thanks, Pavel
|
|
Re: #544: Localization & BeanValidation
Hi Gunnar, sorry, the term "priority" was a bit misleading in this context. Actually it uses the "quality value" from the "Accept-Language" header. Christian 2017-05-29 14:26 GMT+02:00 Gunnar Morling via Groups.Io <gunnar.morling@...>:
Hi Christian, --
|
|