Cases of using AffinityAssignment.clientEventChange method

[Mmuzaf]

Hi Igniters,

Recently I've faced with AffinityAssignment.clientEventChange() method and
not completly sure about the range of its applicability. The javadoc says
"return {@code True} if related discovery event did not cause affinity
assignment change and this assignment is just reference to the previous
one."

Three facts about it:
1) Method is the part of internal Ignite API
2) It​ is​ not used anywhere in Ignite project code
3) "clientEventChage" confusing name for this method. "true" value can be
set not only by client-related events (e.g. cacheChangeRequest,
affinityChangeRequest etc.).

I've prepared diagram when it has "TRUE" value [1].

* Question #1 * When and for what cases we can use returned value of this
method?
* Question #2 * Can it be removed? My suggestion is to keep internal API as
simple as possible.


[1] https://image.ibb.co/cW6Mx8/Client_Event_Change_1.png

[Mmuzaf]

Hello everyone,

Suppose clientEventChange method is useless and have confusing name.

I'm suggesting to remove it as not used. I've filed issue for it:

https://issues.apache.org/jira/browse/IGNITE-8936


чт, 28 июн. 2018 г. в 16:53, Maxim Muzafarov <[hidden email]>:

> Hi Igniters,
>
> Recently I've faced with AffinityAssignment.clientEventChange() method and
> not completly sure about the range of its applicability. The javadoc says
> "return {@code True} if related discovery event did not cause affinity
> assignment change and this assignment is just reference to the previous
> one."
>
> Three facts about it:
> 1) Method is the part of internal Ignite API
> 2) It​ is​ not used anywhere in Ignite project code
> 3) "clientEventChage" confusing name for this method. "true" value can be
> set not only by client-related events (e.g. cacheChangeRequest,
> affinityChangeRequest etc.).
>
> I've prepared diagram when it has "TRUE" value [1].
>
> * Question #1 * When and for what cases we can use returned value of this
> method?
> * Question #2 * Can it be removed? My suggestion is to keep internal API
> as simple as possible.
>
>
> [1] https://image.ibb.co/cW6Mx8/Client_Event_Change_1.png
>

[Dmitriy Pavlov]

Hi Maxim,

Thank you for your efforts to finding this out. I also didn't find any
usages, so I agree here.

I'm going to commit this by lazy consensus if no-one objects within the
next three days.

Sincerely,
Dmitriy Pavlov

чт, 5 июл. 2018 г. в 12:56, Maxim Muzafarov <[hidden email]>:

> Hello everyone,
>
> Suppose clientEventChange method is useless and have confusing name.
>
> I'm suggesting to remove it as not used. I've filed issue for it:
>
> https://issues.apache.org/jira/browse/IGNITE-8936
>
>
> чт, 28 июн. 2018 г. в 16:53, Maxim Muzafarov <[hidden email]>:
>
> > Hi Igniters,
> >
> > Recently I've faced with AffinityAssignment.clientEventChange() method
> and
> > not completly sure about the range of its applicability. The javadoc says
> > "return {@code True} if related discovery event did not cause affinity
> > assignment change and this assignment is just reference to the previous
> > one."
> >
> > Three facts about it:
> > 1) Method is the part of internal Ignite API
> > 2) It​ is​ not used anywhere in Ignite project code
> > 3) "clientEventChage" confusing name for this method. "true" value can be
> > set not only by client-related events (e.g. cacheChangeRequest,
> > affinityChangeRequest etc.).
> >
> > I've prepared diagram when it has "TRUE" value [1].
> >
> > * Question #1 * When and for what cases we can use returned value of this
> > method?
> > * Question #2 * Can it be removed? My suggestion is to keep internal API
> > as simple as possible.
> >
> >
> > [1] https://image.ibb.co/cW6Mx8/Client_Event_Change_1.png
> >
>

[Mmuzaf]

Dmity,

Thank you for reply. I've maked this ticket with "newbie" tag.

I would like to keep this ticket for our new community members. Change is
really simple, so we can easily guide new mebers the development processes
adopted by our community.

Hope this issue will find its author.
If not I'll do it by myself.


пт, 6 июл. 2018 г. в 16:09, Dmitry Pavlov <[hidden email]>:

> Hi Maxim,
>
> Thank you for your efforts to finding this out. I also didn't find any
> usages, so I agree here.
>
> I'm going to commit this by lazy consensus if no-one objects within the
> next three days.
>
> Sincerely,
> Dmitriy Pavlov
>
> чт, 5 июл. 2018 г. в 12:56, Maxim Muzafarov <[hidden email]>:
>
> > Hello everyone,
> >
> > Suppose clientEventChange method is useless and have confusing name.
> >
> > I'm suggesting to remove it as not used. I've filed issue for it:
> >
> > https://issues.apache.org/jira/browse/IGNITE-8936
> >
> >
> > чт, 28 июн. 2018 г. в 16:53, Maxim Muzafarov <[hidden email]>:
> >
> > > Hi Igniters,
> > >
> > > Recently I've faced with AffinityAssignment.clientEventChange() method
> > and
> > > not completly sure about the range of its applicability. The javadoc
> says
> > > "return {@code True} if related discovery event did not cause affinity
> > > assignment change and this assignment is just reference to the previous
> > > one."
> > >
> > > Three facts about it:
> > > 1) Method is the part of internal Ignite API
> > > 2) It​ is​ not used anywhere in Ignite project code
> > > 3) "clientEventChage" confusing name for this method. "true" value can
> be
> > > set not only by client-related events (e.g. cacheChangeRequest,
> > > affinityChangeRequest etc.).
> > >
> > > I've prepared diagram when it has "TRUE" value [1].
> > >
> > > * Question #1 * When and for what cases we can use returned value of
> this
> > > method?
> > > * Question #2 * Can it be removed? My suggestion is to keep internal
> API
> > > as simple as possible.
> > >
> > >
> > > [1] https://image.ibb.co/cW6Mx8/Client_Event_Change_1.png
> > >
> >
>

[Dmitriy Pavlov]

Sute. I like this idea, let's wait.

чт, 12 июл. 2018 г. в 14:48, Maxim Muzafarov <[hidden email]>:

> Dmity,
>
> Thank you for reply. I've maked this ticket with "newbie" tag.
>
> I would like to keep this ticket for our new community members. Change is
> really simple, so we can easily guide new mebers the development processes
> adopted by our community.
>
> Hope this issue will find its author.
> If not I'll do it by myself.
>
>
> пт, 6 июл. 2018 г. в 16:09, Dmitry Pavlov <[hidden email]>:
>
> > Hi Maxim,
> >
> > Thank you for your efforts to finding this out. I also didn't find any
> > usages, so I agree here.
> >
> > I'm going to commit this by lazy consensus if no-one objects within the
> > next three days.
> >
> > Sincerely,
> > Dmitriy Pavlov
> >
> > чт, 5 июл. 2018 г. в 12:56, Maxim Muzafarov <[hidden email]>:
> >
> > > Hello everyone,
> > >
> > > Suppose clientEventChange method is useless and have confusing name.
> > >
> > > I'm suggesting to remove it as not used. I've filed issue for it:
> > >
> > > https://issues.apache.org/jira/browse/IGNITE-8936
> > >
> > >
> > > чт, 28 июн. 2018 г. в 16:53, Maxim Muzafarov <[hidden email]>:
> > >
> > > > Hi Igniters,
> > > >
> > > > Recently I've faced with AffinityAssignment.clientEventChange()
> method
> > > and
> > > > not completly sure about the range of its applicability. The javadoc
> > says
> > > > "return {@code True} if related discovery event did not cause
> affinity
> > > > assignment change and this assignment is just reference to the
> previous
> > > > one."
> > > >
> > > > Three facts about it:
> > > > 1) Method is the part of internal Ignite API
> > > > 2) It​ is​ not used anywhere in Ignite project code
> > > > 3) "clientEventChage" confusing name for this method. "true" value
> can
> > be
> > > > set not only by client-related events (e.g. cacheChangeRequest,
> > > > affinityChangeRequest etc.).
> > > >
> > > > I've prepared diagram when it has "TRUE" value [1].
> > > >
> > > > * Question #1 * When and for what cases we can use returned value of
> > this
> > > > method?
> > > > * Question #2 * Can it be removed? My suggestion is to keep internal
> > API
> > > > as simple as possible.
> > > >
> > > >
> > > > [1] https://image.ibb.co/cW6Mx8/Client_Event_Change_1.png
> > > >
> > >
> >
>