+1 for printing out a warning and ignoring the affinity function from the configuration. There is no other way to 'fix' the configuration other than remove the wrong affinity function, so it can be done at startup time right away.
2016-02-03 9:43 GMT+03:00 Ken Cheng <kcheng....@gmail.com>: > I prefer to throw a IgniteCheckerException. > > Thanks, > kcheng > > On Wed, Feb 3, 2016 at 2:41 PM, Ken Cheng <kcheng....@gmail.com> wrote: > > > Hi Andrey Gura, > > > > > > What's the expected behavior when the cache mode is "Local" but affinity > > function is not "LocalAffinityFunction"? > > > > 1: Throw an exception? > > 2: or change the affinity function rudely as "LocalAffinityFunction" and > > log the warning message at same time? > > > > > > Thanks, > > kcheng > > > > On Wed, Feb 3, 2016 at 10:35 AM, Ken Cheng <kcheng....@gmail.com> wrote: > > > >> Hi Andrey Gura, > >> > >> Thank you very much! I would study this part of code first. > >> > >> Thanks, > >> kcheng > >> > >> On Mon, Feb 1, 2016 at 6:37 PM, Andrey Gura <ag...@gridgain.com> wrote: > >> > >>> Ken, > >>> > >>> cache configuration validation and initialization occurs in > >>> GridCacheProcessor class (methods validate() and initialize()). > >>> > >>> From my point of view two changes should be made: > >>> > >>> - during cache intialization LocalAffinityFunction should be set to > cache > >>> configuration if cache mode is LOCAL; > >>> - warning about ignoring affinity function parameter should be moved > from > >>> validate() method to intialize() method. > >>> > >>> I hope this will help you. > >>> > >>> On Mon, Feb 1, 2016 at 12:12 PM, Ken Cheng <kcheng....@gmail.com> > wrote: > >>> > >>> > Hi Andrey Gura, > >>> > > >>> > I am very new to Ignite, I am going to pick up > >>> > https://issues.apache.org/jira/browse/IGNITE-1481. > >>> > > >>> > Can you please give more hint? > >>> > > >>> > > >>> > > >>> > > >>> > Thanks, > >>> > kcheng > >>> > > >>> > >>> > >>> > >>> -- > >>> Andrey Gura > >>> GridGain Systems, Inc. > >>> www.gridgain.com > >>> > >> > >> > > >
