Re: [infinispan-dev] CacheLoader and CacheStore

On Aug 20, 2013, at 8:14 AM, Tristan Tarrant wrote: >> My preference is for #1. >> >> The main reason is cos JSR-107 is about to hit mainstream and we should try >> to align with it in a way that we reduce confusion. Remember that JSR-107 >> users are potential Infinispan users, and viceversa

Re: [infinispan-dev] CacheLoader and CacheStore

On 08/13/2013 08:59 AM, Galder Zamarreño wrote: > My preference is for #1. > > The main reason is cos JSR-107 is about to hit mainstream and we should try > to align with it in a way that we reduce confusion. Remember that JSR-107 > users are potential Infinispan users, and viceversa, so keeping

Re: [infinispan-dev] CacheLoader and CacheStore

On 08/15/2013 03:48 PM, Galder Zamarreño wrote: > On Aug 14, 2013, at 11:14 PM, Ray Tsang wrote: > >> I actually did not enjoy writing parsers and config builders etc… > ^ Why not? Too verbose? Too many classes? > > I'd rather improve this than carry on supporting property-based configuration > (

Re: [infinispan-dev] CacheLoader and CacheStore

Could we generate the parser and/or XSD as JGroups does with org.jgroups.annotations.Property ? On 15 August 2013 16:52, Ray Tsang wrote: > > > > On Thu, Aug 15, 2013 at 11:45 AM, Radim Vansa wrote: >> >> On 08/15/2013 03:57 PM, Ray Tsang wrote: >> >> >> >> >> On Thu, Aug 15, 2013 at 9:48 AM, G

Re: [infinispan-dev] CacheLoader and CacheStore

On Thu, Aug 15, 2013 at 11:45 AM, Radim Vansa wrote: > On 08/15/2013 03:57 PM, Ray Tsang wrote: > > > > > On Thu, Aug 15, 2013 at 9:48 AM, Galder Zamarreño wrote: > >> >> On Aug 14, 2013, at 11:14 PM, Ray Tsang wrote: >> >> > I actually did not enjoy writing parsers and config builders etc… >>

Re: [infinispan-dev] CacheLoader and CacheStore

On 08/15/2013 03:57 PM, Ray Tsang wrote: On Thu, Aug 15, 2013 at 9:48 AM, Galder Zamarreño > wrote: On Aug 14, 2013, at 11:14 PM, Ray Tsang mailto:saturn...@gmail.com>> wrote: > I actually did not enjoy writing parsers and config builders etc... i felt t

Re: [infinispan-dev] CacheLoader and CacheStore

On 15 Aug 2013, at 14:48, Galder Zamarreño wrote: >> I actually did not enjoy writing parsers and config builders etc… > > ^ Why not? Too verbose? Too many classes? > > I'd rather improve this than carry on supporting property-based configuration > (any other type safety advocates?) Type sa

Re: [infinispan-dev] CacheLoader and CacheStore

On Thu, Aug 15, 2013 at 9:48 AM, Galder Zamarreño wrote: > > On Aug 14, 2013, at 11:14 PM, Ray Tsang wrote: > > > I actually did not enjoy writing parsers and config builders etc… > > i felt that writing the parsers are error prone - there are many places to touch when adding a property - maybe

Re: [infinispan-dev] CacheLoader and CacheStore

On Aug 14, 2013, at 11:14 PM, Ray Tsang wrote: > I actually did not enjoy writing parsers and config builders etc… ^ Why not? Too verbose? Too many classes? I'd rather improve this than carry on supporting property-based configuration (any other type safety advocates?) Cheers, > > On Aug

Re: [infinispan-dev] CacheLoader and CacheStore

On 08/14/2013 08:58 PM, Mircea Markus wrote: > On 14 Aug 2013, at 18:48, Dennis Reed wrote: > >>> I've written such parser on Friday, basically just copying and rewriting >>> the SingleFileCacheStore stuff (adding my own properties instead). It >>> took me about an hour and I don't hate anyone fro

Re: [infinispan-dev] CacheLoader and CacheStore

I actually did not enjoy writing parsers and config builders etc... On Aug 14, 2013, at 14:58, Mircea Markus wrote: > > On 14 Aug 2013, at 18:48, Dennis Reed wrote: > >>> I've written such parser on Friday, basically just copying and rewriting >>> the SingleFileCacheStore stuff (adding my own p

Re: [infinispan-dev] CacheLoader and CacheStore

On 14 Aug 2013, at 18:48, Dennis Reed wrote: >> I've written such parser on Friday, basically just copying and rewriting >> the SingleFileCacheStore stuff (adding my own properties instead). It >> took me about an hour and I don't hate anyone from infinispan team :) >> Maybe I am already advance

Re: [infinispan-dev] CacheLoader and CacheStore

On 08/14/2013 04:21 AM, Radim Vansa wrote: > On 08/13/2013 04:50 PM, Mircea Markus wrote: >> On 13 Aug 2013, at 15:30, Galder Zamarreño wrote: >> >>> On Aug 13, 2013, at 12:20 PM, Mircea Markus wrote: >>> On 13 Aug 2013, at 07:59, Galder Zamarreño wrote: > My preference is for #1.

Re: [infinispan-dev] CacheLoader and CacheStore

On 14 Aug 2013, at 11:07, Mircea Markus wrote: > > On 14 Aug 2013, at 10:21, Radim Vansa wrote: > >> On 08/13/2013 04:50 PM, Mircea Markus wrote: >>> On 13 Aug 2013, at 15:30, Galder Zamarreño wrote: >>> On Aug 13, 2013, at 12:20 PM, Mircea Markus wrote: > On 13 Aug 2013, at

Re: [infinispan-dev] CacheLoader and CacheStore

On 14 Aug 2013, at 10:21, Radim Vansa wrote: > On 08/13/2013 04:50 PM, Mircea Markus wrote: >> On 13 Aug 2013, at 15:30, Galder Zamarreño wrote: >> >>> On Aug 13, 2013, at 12:20 PM, Mircea Markus wrote: >>> On 13 Aug 2013, at 07:59, Galder Zamarreño wrote: > My preference is

Re: [infinispan-dev] CacheLoader and CacheStore

On 08/13/2013 04:50 PM, Mircea Markus wrote: > On 13 Aug 2013, at 15:30, Galder Zamarreño wrote: > >> On Aug 13, 2013, at 12:20 PM, Mircea Markus wrote: >> >>> On 13 Aug 2013, at 07:59, Galder Zamarreño wrote: >>> My preference is for #1. The main reason is cos JSR-107 is about to

Re: [infinispan-dev] CacheLoader and CacheStore

On 13 Aug 2013, at 15:30, Galder Zamarreño wrote: > On Aug 13, 2013, at 12:20 PM, Mircea Markus wrote: > >> On 13 Aug 2013, at 07:59, Galder Zamarreño wrote: >> >>> My preference is for #1. >>> >>> The main reason is cos JSR-107 is about to hit mainstream and we should try >>> to align wi

Re: [infinispan-dev] CacheLoader and CacheStore

er+class configuration… >> >> [6] >> https://docs.jboss.org/author/display/ISPN/Cache+Loaders+and+Stores#CacheLoadersandStores-Filesystembasedcacheloaders >> >>> Most of the users need read/write operations. So they'd implement a Loader >>> and a Writer and then configure it as a

Re: [infinispan-dev] CacheLoader and CacheStore

asedcacheloaders > >> Most of the users need read/write operations. So they'd implement a Loader >> and a Writer and then configure it as a Persistence. (#1) >> Or they can implement a Loader and configure it as a Loader (#2) ;) >> >> >>> >>> -

Re: [infinispan-dev] CacheLoader and CacheStore

need read/write operations. So they'd implement a Loader > and a Writer and then configure it as a Persistence. (#1) > Or they can implement a Loader and configure it as a Loader (#2) ;) > > >> >> - Original Message ----- >> From: "Mircea Markus&q

Re: [infinispan-dev] CacheLoader and CacheStore

ot;Mircea Markus" > To: "infinispan -Dev List" > Sent: Friday, 9 August, 2013 7:21:14 PM > Subject: [infinispan-dev] CacheLoader and CacheStore > > Hi, > > Apologies for the long email :-) > > There have been several discussions around how the CacheStore a

Re: [infinispan-dev] CacheLoader and CacheStore

Subject: [infinispan-dev] CacheLoader and CacheStore Hi, Apologies for the long email :-) There have been several discussions around how the CacheStore and CacheLoader functionality should look in the new CacheLoader API. Here's are the possible approaches: 1. Have CacheLoader and C

Re: [infinispan-dev] CacheLoader and CacheStore

My vote is #2, as it simplifies the code and removes redundant code (fewer chances for bugs). We've already had to fix one bug where the code reading the cache loader configuration was different than reading the cache store configuration. -Dennis On 08/09/2013 01:21 PM, Mircea Markus wrote: >

[infinispan-dev] CacheLoader and CacheStore

Hi, Apologies for the long email :-) There have been several discussions around how the CacheStore and CacheLoader functionality should look in the new CacheLoader API. Here's are the possible approaches: 1. Have CacheLoader and CacheWriter as independent interfaces, the way JSR 107 does it ([