Re: Async methods for JAX-RS WebClient

2012-09-24 Thread Sergey Beryozkin
I went ahead and populated AsyncInvoker, as well as added another shortcut directly to WebClient, to support an async post call. Dan, have a look please, One thing that I can have a look later is making sure the retry calls work in case of async call failures, plus add few more minor

Re: Async methods for JAX-RS WebClient

2012-09-24 Thread Daniel Kulp
On Sep 24, 2012, at 8:31 AM, Sergey Beryozkin sberyoz...@gmail.com wrote: I went ahead and populated AsyncInvoker, as well as added another shortcut directly to WebClient, to support an async post call. Dan, have a look please, That all looks great. Thanks for filling in all the other

Re: Async methods for JAX-RS WebClient

2012-09-24 Thread Sergey Beryozkin
On 24/09/12 19:07, Daniel Kulp wrote: On Sep 24, 2012, at 8:31 AM, Sergey Beryozkinsberyoz...@gmail.com wrote: I went ahead and populated AsyncInvoker, as well as added another shortcut directly to WebClient, to support an async post call. Dan, have a look please, That all looks great.

Re: Async methods for JAX-RS WebClient

2012-09-23 Thread Sergey Beryozkin
Hi Dan On 21/09/12 19:27, Daniel Kulp wrote: Sergey, (and others) I just committed some initial support for some async methods to the WebClient. Can you take a look at that change and make sure it all makes sense? I only have a get method in there right now, but it should be fairly

Re: Async methods for JAX-RS WebClient

2012-09-23 Thread Sergey Beryozkin
On 23/09/12 18:14, Sergey Beryozkin wrote: Hi Dan On 21/09/12 19:27, Daniel Kulp wrote: Sergey, (and others) I just committed some initial support for some async methods to the WebClient. Can you take a look at that change and make sure it all makes sense? I only have a get method in there

Async methods for JAX-RS WebClient

2012-09-21 Thread Daniel Kulp
Sergey, (and others) I just committed some initial support for some async methods to the WebClient. Can you take a look at that change and make sure it all makes sense? I only have a get method in there right now, but it should be fairly trivial now to add the others that would map to