ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dmitriy Setrakyan <dsetrak...@apache.org>
Subject Re: AffinityKeyMapper: break compatibility before 2.0
Date Tue, 11 Apr 2017 07:41:17 GMT
I agree that this interface is problematic. However, I don't think that
dropping it right away would be fair to our users. Can we deprecate it and
print out a warning that AffinityKeyMapper cannot be used with DDL


On Tue, Apr 11, 2017 at 12:32 AM, Sergi Vladykin <sergi.vladykin@gmail.com>

> Guys,
> We are moving in direction of better distributed SQL support, it means that
> we always will need to know an affinity field name for key type.
> Now we have AffinityKeyMapper which hides it from us.
> Since we have other means of configuring the affinity key, e.g
> CacheKeyConfiguration and @AffinityKeyMapped, I suggest to either drop this
> interface or change it so that it just return affinity key field name
> instead of the affinity key field value. I prefer dropping it.
> What do you think?
> Sergi

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