ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dmitriy Setrakyan <dsetrak...@apache.org>
Subject Re: Deprecate CacheConfiguration.setIndexedTypes
Date Thu, 20 Apr 2017 00:02:20 GMT
Vladimir, I am not sure I understand the issue here. Why would the Example
1 result in an exception?

On Wed, Apr 19, 2017 at 1:22 PM, Vladimir Ozerov <vozerov@gridgain.com>
wrote:

> Igniters,
>
> I'd like to propose to deprecate infamous CacheConfiguration.setIndexedT
> ypes
> 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