Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-26 Thread Galder Zamarreño
On 19 Feb 2014, at 12:03, Sanne Grinovero sa...@infinispan.org wrote: On 19 February 2014 07:12, Galder Zamarreño gal...@redhat.com wrote: On 03 Feb 2014, at 19:01, Dan Berindei dan.berin...@gmail.com wrote: On Mon, Feb 3, 2014 at 6:28 PM, Radim Vansa rva...@redhat.com wrote: For

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-26 Thread Galder Zamarreño
On 19 Feb 2014, at 17:44, Dennis Reed der...@redhat.com wrote: On 02/19/2014 12:57 AM, Galder Zamarreño wrote: On 31 Jan 2014, at 08:32, Dennis Reed der...@redhat.com wrote: It would be a loss of functionality. As a common example, the AS web session replication cache is configured for

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-26 Thread Dan Berindei
On Wed, Feb 26, 2014 at 8:56 AM, Galder Zamarreño gal...@redhat.com wrote: On 19 Feb 2014, at 12:03, Sanne Grinovero sa...@infinispan.org wrote: On 19 February 2014 07:12, Galder Zamarreño gal...@redhat.com wrote: On 03 Feb 2014, at 19:01, Dan Berindei dan.berin...@gmail.com wrote:

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-19 Thread Galder Zamarreño
On 31 Jan 2014, at 08:32, Dennis Reed der...@redhat.com wrote: It would be a loss of functionality. As a common example, the AS web session replication cache is configured for ASYNC by default, for performance reasons. But it can be changed to SYNC to guarantee that when the request

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-19 Thread Galder Zamarreño
On 03 Feb 2014, at 19:01, Dan Berindei dan.berin...@gmail.com wrote: On Mon, Feb 3, 2014 at 6:28 PM, Radim Vansa rva...@redhat.com wrote: For sync we would want to invoke directly to avoid context switching. I think you haven't properly understood what I was talking about: the

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-19 Thread Galder Zamarreño
On 31 Jan 2014, at 11:59, Sanne Grinovero sa...@infinispan.org wrote: Generally I like the systems designed with SYNC_DIST + async shared cachestore. It's probably the best setup we can offer: - you need a shared cachestore for persistence consistency - using SYNC distribution to other

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-19 Thread Dan Berindei
On Wed, Feb 19, 2014 at 1:03 PM, Sanne Grinovero sa...@infinispan.orgwrote: On 19 February 2014 07:12, Galder Zamarreño gal...@redhat.com wrote: On 03 Feb 2014, at 19:01, Dan Berindei dan.berin...@gmail.com wrote: On Mon, Feb 3, 2014 at 6:28 PM, Radim Vansa rva...@redhat.com wrote:

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-19 Thread Dennis Reed
On 02/19/2014 12:57 AM, Galder Zamarreño wrote: On 31 Jan 2014, at 08:32, Dennis Reed der...@redhat.com wrote: It would be a loss of functionality. As a common example, the AS web session replication cache is configured for ASYNC by default, for performance reasons. But it can be changed to

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-03 Thread Radim Vansa
See below... On Fri, Jan 31, 2014 at 7:35 AM, Radim Vansa rva...@redhat.com wrote: Worth to note that Infinispan does not have true async operation - executing synchronous request in another threadpool is rather simplistic solution that has serious drawbacks (I can imagine a situation where

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-03 Thread Sanne Grinovero
On 3 February 2014 14:10, Radim Vansa rva...@redhat.com wrote: See below... On Fri, Jan 31, 2014 at 7:35 AM, Radim Vansa rva...@redhat.com wrote: Worth to note that Infinispan does not have true async operation - executing synchronous request in another threadpool is rather simplistic

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-03 Thread William Burns
On Mon, Feb 3, 2014 at 9:54 AM, Sanne Grinovero sa...@infinispan.org wrote: On 3 February 2014 14:10, Radim Vansa rva...@redhat.com wrote: See below... On Fri, Jan 31, 2014 at 7:35 AM, Radim Vansa rva...@redhat.com wrote: Worth to note that Infinispan does not have true async operation -

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-03 Thread Radim Vansa
For sync we would want to invoke directly to avoid context switching. I think you haven't properly understood what I was talking about: the putAsync should not switch context at all in the ideal design. It should traverse through the interceptors all the way down (logically, in current

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-02 Thread Bela Ban
At the JGroups level, ASYNC generates *less* traffic than SYNC. So if you do sync under the cover and use a future to make it async at the API level, you're incurring more overhead, namely the messages sending back the responses. Not sure about the Infinispan async API, but I'd assume this

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-02-02 Thread Bela Ban
On 31/01/14 13:35, Radim Vansa wrote: Worth to note that Infinispan does not have true async operation - executing synchronous request in another threadpool is rather simplistic solution that has serious drawbacks (I can imagine a situation where I'd do 100 async gets in parallel, but this

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-01-31 Thread Tristan Tarrant
Couldn't this be handled higher up in our implementatoin then ? If I enable an async mode, all puts / gets become putAsync/getAsync transparently to both the application and to the state transfer. Tristan On 01/31/2014 08:32 AM, Dennis Reed wrote: It would be a loss of functionality. As a

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-01-31 Thread Sanne Grinovero
Generally I like the systems designed with SYNC_DIST + async shared cachestore. It's probably the best setup we can offer: - you need a shared cachestore for persistence consistency - using SYNC distribution to other replicas provides a fairly decent resilience - if your cachestore needs to be

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-01-31 Thread William Burns
+1 to moving to the async methods only. I have mentioned this as well in passing when discussing L1 as there is no way to ensure consistency with an async transport. Although if we fire the async methods with either SKIP_REMOTE_LOOKUP/IGNORE_RETURN_VALUES flag than this consistency is still lost

[infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-01-30 Thread Galder Zamarreño
Hi all, The following came to my mind yesterday: I think we should ditch ASYNC modes for DIST/REPL/INV and our async cache store functionality. Instead, whoever wants to store something asyncronously should use asynchronous methods, i.e. call putAsync. So, this would mean that when you call

Re: [infinispan-dev] Ditching ASYNC modes for REPL/DIST/INV/CacheStores?

2014-01-30 Thread Dennis Reed
It would be a loss of functionality. As a common example, the AS web session replication cache is configured for ASYNC by default, for performance reasons. But it can be changed to SYNC to guarantee that when the request finishes that the session was replicated. That wouldn't be possible if