Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CAGcMBHgmy_-SOC2+xd9xQiw4_eWXOSB_hN2_zWQ9ZBwp1Tfz8w@mail.gmail.com>
References: 
 <CAPJKmXmXW_F_ButZ=xOX68cpj=gWa=w077NN+BX0GfpGUi9ETA@mail.gmail.com>
 <CA+0=VoUKmm1B2i3KaxHVPrOF9wZ4d8LurYeYC=8fqDC7yBOzZA@mail.gmail.com>
 <CAPJKmXmh2gmq1chYQpv3T2K8yvQcZVbuDc_kqhHhJ-PO+1n3QA@mail.gmail.com>
 <CAKCfC3_THC8FX4u0TZ20z9ng9pF_CG4-KU=9_=RHCfCoKu9F2Q@mail.gmail.com>
 <CAGcMBHgmy_-SOC2+xd9xQiw4_eWXOSB_hN2_zWQ9ZBwp1Tfz8w@mail.gmail.com>
From: Dmitriy Setrakyan <dsetrakyan@apache.org>
Date: Tue, 16 Feb 2016 02:15:42 -0800
Message-ID: 
 <CA+0=VoV-P17AwQofMibtkexc79VY7Bsu-CA3t2JiYo+pZYB0jg@mail.gmail.com>
Subject: Re: NoopSwapSpaceSpi and dynamically started caches
To: dev@ignite.apache.org
Content-Type: multipart/alternative; boundary=089e013a0996cc709d052be06c26

--089e013a0996cc709d052be06c26
Content-Type: text/plain; charset=UTF-8

Sounds good to me.

On Tue, Feb 16, 2016 at 2:13 AM, Yakov Zhdanov <yzhdanov@apache.org> wrote:

> Let's require that all nodes should have same swap space SPI. I think we
> can put @GridSpiConsistencyChecked on file swap space SPI (any other
> suggestions?).
>
> --Yakov
>
> 2016-02-15 23:31 GMT+03:00 Konstantin Margorin <ruskim@gmail.com>:
>
> > Probably I can handle this ticket, but I want to clarify one thing.
> >
> > Suppose dynamic cache with enabled swap should be partitioned between two
> > nodes with backups=1. First node started with FileSwapSpaceSpi, second
> with
> > NoopSwapSpaceSpi.
> >
> > How cache should start in this case?
> >
> >
> > On Mon, Feb 15, 2016 at 3:45 PM, Artem Shutak <ashutak@gridgain.com>
> > wrote:
> >
> > > I've created a newbie issue for that:
> > > https://issues.apache.org/jira/browse/IGNITE-2650.
> > >
> > > -- Artem --
> > >
> > > On Wed, Feb 10, 2016 at 8:53 PM, Dmitriy Setrakyan <
> > dsetrakyan@apache.org>
> > > wrote:
> > >
> > > > Artem,
> > > >
> > > > I like the option with proper exception in case if a dynamic cache is
> > > > started with swap enabled while there was no swap configured in
> Ignite.
> > > >
> > > > Starting FileSwapSpaceSpi all the time is not a good option as user
> > will
> > > > face the same problem when trying to change to a different SPI.
> > > >
> > > > D.
> > > >
> > > > On Wed, Feb 10, 2016 at 4:39 AM, Artem Shutak <ashutak@gridgain.com>
> > > > wrote:
> > > >
> > > > > Igniters,
> > > > >
> > > > > I've noticed an issue with NoopSwapSpaceSpi and dynamically started
> > > > cashes.
> > > > >
> > > > > We have the following logic on node start:
> > > > >
> > > > >             if (cfg.getSwapSpaceSpi() == null) {
> > > > >                 boolean needSwap = false;
> > > > >
> > > > >                 if (cfg.getCacheConfiguration() != null &&
> > > > > !Boolean.TRUE.equals(cfg.isClientMode())) {
> > > > >                     for (CacheConfiguration c :
> > > > > cfg.getCacheConfiguration()) {
> > > > >                         if (c.isSwapEnabled()) {
> > > > >                             needSwap = true;
> > > > >
> > > > >                             break;
> > > > >                         }
> > > > >                     }
> > > > >                 }
> > > > >
> > > > >                 cfg.setSwapSpaceSpi(needSwap ? new
> > FileSwapSpaceSpi() :
> > > > new
> > > > > NoopSwapSpaceSpi());
> > > > >             }
> > > > >
> > > > > According to the code above, if IgniteConfiguration doesn't contain
> > > cache
> > > > > configurations or all configured caches configured without swap,
> then
> > > > node
> > > > > will be started with NoopSwapSpaceSpi.
> > > > > As a result in this case, when we start dynamic cache with enabled
> > swap
> > > > > then actually swap doesn't work.
> > > > >
> > > > > There is simple workaround - user have to explicitly set
> > > FileSwapSpaceSpi
> > > > > to configuration. But now, there is no any warning or exception.
> > > > >
> > > > > What we can do:
> > > > >
> > > > >    1. Throw an exception on dynamic cache start (with swap enabled)
> > > > >    if NoopSwapSpaceSpi is configured.
> > > > >    2. Actually, I don't see why we need NoopSwapSpaceSpi at all. I
> > > think
> > > > we
> > > > >    should configure FileSwapSpaceSpi always if user doesn't provide
> > any
> > > > >    SwapSpaceSpi explicitly.
> > > > >
> > > > > Thoughts?
> > > > >
> > > > > Thanks,
> > > > > -- Artem --
> > > > >
> > > >
> > >
> >
>

--089e013a0996cc709d052be06c26--