Re: Deprecate CacheConfiguration.setIndexedTypes

2017-04-20 Thread Vladimir Ozerov
ss, Value2.class); > > > => exeption > > > > > > Example 2 > > > setIndexedTypes(Key1.class, Value1.class); > > > setQueryEntitities(new QueryEntity(...)); > > > => all ok, two query entities > > > > > > Proposal: > > > 1) Deprecate CacheConfiguration.setIndexedTypes > > > 2) Add new constructor QueryEntity(Class keyCls, Class valCls) > > > 3) Remove all non-obvious semanrics from > > > CacheConfiguration.setQueryEntities method > > > and make it plain setter. > > > > > > Thoughts? > > > > > >

Re: Deprecate CacheConfiguration.setIndexedTypes

2017-04-20 Thread Valentin Kulichenko
ss); > > => exeption > > > > Example 2 > > setIndexedTypes(Key1.class, Value1.class); > > setQueryEntitities(new QueryEntity(...)); > > => all ok, two query entities > > > > Proposal: > > 1) Deprecate CacheConfiguration.setIndexedTypes > > 2) Add new co

Re: Deprecate CacheConfiguration.setIndexedTypes

2017-04-19 Thread Dmitriy Setrakyan
alue1.class); > setQueryEntitities(new QueryEntity(...)); > => all ok, two query entities > > Proposal: > 1) Deprecate CacheConfiguration.setIndexedTypes > 2) Add new constructor QueryEntity(Class keyCls, Class valCls) > 3) Remove all non-obvious semanrics from > CacheConfiguration.setQueryEntities method > and make it plain setter. > > Thoughts? >

Re: Deprecate CacheConfiguration.setIndexedTypes

2017-04-19 Thread Denis Magda
t; exeption > > Example 2 > setIndexedTypes(Key1.class, Value1.class); > setQueryEntitities(new QueryEntity(...)); > => all ok, two query entities > > Proposal: > 1) Deprecate CacheConfiguration.setIndexedTypes > 2) Add new constructor QueryEnt

Deprecate CacheConfiguration.setIndexedTypes

2017-04-19 Thread Vladimir Ozerov
class); => exeption Example 2 setIndexedTypes(Key1.class, Value1.class); setQueryEntitities(new QueryEntity(...)); => all ok, two query entities Proposal: 1) Deprecate CacheConfiguration.setIndexedTypes 2) Add new constructor QueryEntity(Class keyCls, Class valCls) 3) Remove all non-obvious sem