Re: [hibernate-dev] [OGM] The (new) remote Hot-Rod based Infinispan GridDialect

2015-12-14 Thread Sanne Grinovero
On 14 December 2015 at 18:09, Gunnar Morling wrote: > Does it have to be a separate module to begin with? > > For MongoDB - which contains two datastore providers (MongoDB, Fongo) > and Redis - which also will have two different dialects as per Mark's > pending PR - it's one module. > > We should

Re: [hibernate-dev] [OGM] The (new) remote Hot-Rod based Infinispan GridDialect

2015-12-14 Thread Davide D'Alto
Sorry for the previous email, I've sent it too soon pressing some strange keywords combination. > We should stick to one pattern, and having one module seems easier on > the user to me. So unless you see a strong advantage for two modules > I'd say let's use one. One general disadvantage I can se

Re: [hibernate-dev] [OGM] The (new) remote Hot-Rod based Infinispan GridDialect

2015-12-14 Thread Gunnar Morling
Does it have to be a separate module to begin with? For MongoDB - which contains two datastore providers (MongoDB, Fongo) and Redis - which also will have two different dialects as per Mark's pending PR - it's one module. We should stick to one pattern, and having one module seems easier on the u

Re: [hibernate-dev] [OGM] The (new) remote Hot-Rod based Infinispan GridDialect

2015-12-14 Thread Davide D'Alto
I would use: hibernate-ogm-infinispan-hotrod For Neo4j we have a similart problem at the moment the directory is hibernate-ogm-neo4j and the short name is 'neo4j_embedded' Whatever we choose I guess we should be consistent: I guess haivng - hibernate-ogm-infinispan/embedded - hibernate-ogm-infin

[hibernate-dev] [OGM] The (new) remote Hot-Rod based Infinispan GridDialect

2015-12-14 Thread Sanne Grinovero
Hello all, while creating the basic scaffolding for the new GridDialect, I called the new Maven module "hibernate-ogm-infinispan-hotrod". Which is rather long, but descriptive. Q1: any better name? The current one which we have working on Infinispan "embedded mode" is named "hibernate-ogm-