Re: AffinityKeyMapper alternatives
Dmitry, IMO, it's actually pretty typical for data grid use cases where affinity key is usually provided as part of key itself, i.e. after cache creation. In vast majority of cases I've seen, this is done via very popular @AffinityKeyMapped annotation. My only point is that the annotation can't always be used due to restrictions on application level. For example, an application I was recently working with has a platform that uses Ignite internally and provides its own API. Therefore it can't expose @AffinityKeyMapped to its users, but instead provides another annotation (and probably some other custom mechanisms). It currently uses deprecated AffinityKeyMapper to support this and there is no alternative. Just getting rid of it would mean changing and complicating the platform API. -Val On Mon, Sep 11, 2017 at 6:25 PM Dmitriy Setrakyanwrote: > On Mon, Sep 11, 2017 at 4:47 PM, Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > > > Even if CacheKeyConfiguration is part of CacheConfiguration, the affinity > > key field name can be provided only on cache startup. In many cases this > > name can be resolved only based on the actual key instance, e.g. during > > first put. Per my understanding, this already works with annotation, I > just > > propose more flexible solution for rare cases when annotation can't be > > used. Basically, the logic we currently have would become the default > > implementation of the resolver. > > > > Val, the use case seems very strange to me. How can you not know the > affinity key field in advance? Can you provide an example from the field? >
Re: AffinityKeyMapper alternatives
On Mon, Sep 11, 2017 at 4:47 PM, Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > Even if CacheKeyConfiguration is part of CacheConfiguration, the affinity > key field name can be provided only on cache startup. In many cases this > name can be resolved only based on the actual key instance, e.g. during > first put. Per my understanding, this already works with annotation, I just > propose more flexible solution for rare cases when annotation can't be > used. Basically, the logic we currently have would become the default > implementation of the resolver. > Val, the use case seems very strange to me. How can you not know the affinity key field in advance? Can you provide an example from the field?
Re: AffinityKeyMapper alternatives
Even if CacheKeyConfiguration is part of CacheConfiguration, the affinity key field name can be provided only on cache startup. In many cases this name can be resolved only based on the actual key instance, e.g. during first put. Per my understanding, this already works with annotation, I just propose more flexible solution for rare cases when annotation can't be used. Basically, the logic we currently have would become the default implementation of the resolver. -Val On Mon, Sep 11, 2017 at 4:41 PM, Dmitriy Setrakyanwrote: > On Mon, Sep 11, 2017 at 4:01 PM, Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > > > Guys, > > > > Some time ago we deprecated AffinityKeyMapper in favor > > of CacheKeyConfiguration#affinityKeyFieldName and AffinityKeyMapped > > annotation. While I understand the reasons why we did this, I think it's > > not very flexible as requires to specify the field name on node startup. > > > > First of all, CacheKeyConfiguration is set on IgniteConfiguration, but > not > > CacheConfiguration. Does anyone knows why? How can I specify the affinity > > field name if I create a new cache dynamically? > > > > Ouch... really? Just looking at the name of the configuration class, I > would assume that it belongs to CacheConfiguration. Can this be fixed? > > > > > > Second of all, AffinityKeyMapped doesn't always work. There are cases > when > > model classes can't be modified with Ignite annotations, for example. For > > this case I suggest to introduce something like > > AffinityKeyFieldNameResolver that will allow to implement custom logic > > instead. Of course, it will work in the same way as annotation, i.e. > > invoked on client side only. Is this possible? > > > > But wouldn't CacheKeyConfiguration provide this information if it was the > property of the CacheConfiguration and not IgniteConfiguration? I don't > think we need a resolver. >
Re: AffinityKeyMapper alternatives
On Mon, Sep 11, 2017 at 4:01 PM, Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > Guys, > > Some time ago we deprecated AffinityKeyMapper in favor > of CacheKeyConfiguration#affinityKeyFieldName and AffinityKeyMapped > annotation. While I understand the reasons why we did this, I think it's > not very flexible as requires to specify the field name on node startup. > > First of all, CacheKeyConfiguration is set on IgniteConfiguration, but not > CacheConfiguration. Does anyone knows why? How can I specify the affinity > field name if I create a new cache dynamically? > Ouch... really? Just looking at the name of the configuration class, I would assume that it belongs to CacheConfiguration. Can this be fixed? > > Second of all, AffinityKeyMapped doesn't always work. There are cases when > model classes can't be modified with Ignite annotations, for example. For > this case I suggest to introduce something like > AffinityKeyFieldNameResolver that will allow to implement custom logic > instead. Of course, it will work in the same way as annotation, i.e. > invoked on client side only. Is this possible? > But wouldn't CacheKeyConfiguration provide this information if it was the property of the CacheConfiguration and not IgniteConfiguration? I don't think we need a resolver.
AffinityKeyMapper alternatives
Guys, Some time ago we deprecated AffinityKeyMapper in favor of CacheKeyConfiguration#affinityKeyFieldName and AffinityKeyMapped annotation. While I understand the reasons why we did this, I think it's not very flexible as requires to specify the field name on node startup. First of all, CacheKeyConfiguration is set on IgniteConfiguration, but not CacheConfiguration. Does anyone knows why? How can I specify the affinity field name if I create a new cache dynamically? Second of all, AffinityKeyMapped doesn't always work. There are cases when model classes can't be modified with Ignite annotations, for example. For this case I suggest to introduce something like AffinityKeyFieldNameResolver that will allow to implement custom logic instead. Of course, it will work in the same way as annotation, i.e. invoked on client side only. Is this possible? -Val