Re: [JCS] JCache CDI integration

2017-09-03 Thread Romain Manni-Bucau
Almost had the same except when using a lot of keys where a single lock is an issue or a plain map which is still 4 times faster under a medium load just doing gets (50 threads). Issue is we ignore if the underlying memcache is thread safe which means we could avoid the composite cache synchronizat

Re: [JCS] JCache CDI integration

2017-09-03 Thread Thomas Vandahl
On 02.09.17 10:41, Romain Manni-Bucau wrote: > Ok, got the confirmation for the reflection fix. > > Now we lock in CompositeCache.get. Wonder if we could have a lock free > MemoryCache implementation, at least for read side of things. Sounds doable > using ConcurrentMap like algorithms but can req

Re: [JCS] JCache CDI integration

2017-09-02 Thread Romain Manni-Bucau
Ok, got the confirmation for the reflection fix. Now we lock in CompositeCache.get. Wonder if we could have a lock free MemoryCache implementation, at least for read side of things. Sounds doable using ConcurrentMap like algorithms but can require more time than I have ATM to validate it :(. In ot

Re: [JCS] JCache CDI integration

2017-09-01 Thread Gary Gregory
Well, you know me, I like RERO! :-) G On Fri, Sep 1, 2017 at 9:32 AM, Romain Manni-Bucau wrote: > I'm waiting for some confirmation on a complex app but it can likely be the > case in ~1 month if nobody does it before. > > > Romain Manni-Bucau > @rmannibucau |

Re: [JCS] JCache CDI integration

2017-09-01 Thread Romain Manni-Bucau
I'm waiting for some confirmation on a complex app but it can likely be the case in ~1 month if nobody does it before. Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github

Re: [JCS] JCache CDI integration

2017-09-01 Thread Gary Gregory
Do you plan on making a release for push this out? Gary On Fri, Sep 1, 2017 at 9:21 AM, Romain Manni-Bucau wrote: > Hi guys, > > Due to some performance issues due to the reflection usage in the JCache > CDI integration we have, I pushed some changes in that layer. > > It is not expected to cha

[JCS] JCache CDI integration

2017-09-01 Thread Romain Manni-Bucau
Hi guys, Due to some performance issues due to the reflection usage in the JCache CDI integration we have, I pushed some changes in that layer. It is not expected to change anything functionally but since the changes were quite more deep than I expected don't hesitate to shout if you observe some