ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vladimir Ozerov <voze...@gridgain.com>
Subject Deprecate CacheConfiguration.setIndexedTypes
Date Wed, 19 Apr 2017 20:22:38 GMT
Igniters,

I'd like to propose to deprecate infamous CacheConfiguration.setIndexedTypes
method. It has subtle semantics and clashes with
CacheConfiguration.setQueryEntities method. Several examples.

Example 1
setIndexedTypes(Key1.class, Value1.class);
setIndexedTypes(Key2.class, 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?

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message