Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.apache.org
From: Kozlov Maxim <dreamx.max@gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
Subject: Re: Add isPrimary() and isBackup() methods on CacheQueryEntryEvent
Date: Wed, 9 Aug 2017 16:49:44 +0300
References: <E9437F56-E366-4AC1-954C-F42E0DBEFFC4@gmail.com>
 <CAP3QaUGcCVHGbOxttkH2_PUGfj2edkaZDEHy4-GqVF1EkKVYAg@mail.gmail.com>
 <197DA001-1B67-4B21-9A4C-5CD0BD0121BC@gmail.com>
 <CAOKDqC_9dnYDxpji0V3TdJjyR5dwLRvA9hZuRxMGChfM_yGQjA@mail.gmail.com>
 <5A8431D2-2EBD-4070-932A-2E5FEBB261B1@gmail.com>
To: dev@ignite.apache.org
In-Reply-To: <5A8431D2-2EBD-4070-932A-2E5FEBB261B1@gmail.com>
Message-Id: <C754429E-6400-453E-9382-15537975E70F@gmail.com>
archived-at: Wed, 09 Aug 2017 13:49:51 -0000

Igniters,

What do you think about the new API?


> 3 =D0=B0=D0=B2=D0=B3. 2017 =D0=B3., =D0=B2 15:56, Kozlov Maxim =
<dreamx.max@gmail.com> =D0=BD=D0=B0=D0=BF=D0=B8=D1=81=D0=B0=D0=BB(=D0=B0):=

>=20
> Guys,=20
>=20
> I'm sorry for the misunderstanding, I was tired at the end of the day =
:-)
> In the process of working on the task, I had to add 2 methods to the =
public interface.
> The first method is #isBackup(), it returns 'true' if cache is being =
updated on the backup node.
> The second method is #isPrimary(), it returns 'true' if cache is being =
updated on the primary node.
> Their main purpose is to show where a continuous query filter has been =
invoked.
>=20
> Any thoughts about such solution?
>=20
>> 3 =D0=B0=D0=B2=D0=B3. 2017 =D0=B3., =D0=B2 14:45, Anton Vinogradov =
<avinogradov@gridgain.com> =D0=BD=D0=B0=D0=BF=D0=B8=D1=81=D0=B0=D0=BB(=D0=B0=
):
>>=20
>> Folks,
>>=20
>> As far as I see, Issue still in PatchAvailable state, what did you =
mean by
>> "solved"?
>>=20
>> On Wed, Aug 2, 2017 at 8:01 PM, Kozlov Maxim <dreamx.max@gmail.com> =
wrote:
>>=20
>>> Sure.
>>>=20
>>> CacheQueryEntryEvent:
>>>=20
>>> public abstract boolean isBackup();
>>> public abstract boolean isPrimary();
>>>=20
>>>=20
>>>> 2 =D0=B0=D0=B2=D0=B3. 2017 =D0=B3., =D0=B2 19:56, Nikolai Tikhonov =
<ntikhonov@apache.org>
>>> =D0=BD=D0=B0=D0=BF=D0=B8=D1=81=D0=B0=D0=BB(=D0=B0):
>>>>=20
>>>> Max,
>>>>=20
>>>> Thank you for your contribution! Could you share here what exactly =
was
>>>> added to interface?
>>>>=20
>>>> On Wed, Aug 2, 2017 at 7:53 PM, Kozlov Maxim <dreamx.max@gmail.com>
>>> wrote:
>>>>=20
>>>>> Igniters,
>>>>>=20
>>>>> When you solved the 3878[1] ticket, two methods were added[2]:
>>> isPrimary()
>>>>> and isBackup() on the CacheQueryEntryEvent in a public API. Do you =
agree
>>>>> with this decision?
>>>>>=20
>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-3878 <
>>>>> https://issues.apache.org/jira/browse/IGNITE-3878>
>>>>> [2] https://github.com/apache/ignite/pull/1393 <
>>> https://github.com/apache/
>>>>> ignite/pull/1393>
>>>>>=20
>>>>> --
>>>>> Best Regards,
>>>>> Max K.
>>>>>=20
>>>>>=20
>>>>>=20
>>>>>=20
>>>>>=20
>>>=20
>>> --
>>> Best Regards,
>>> Max K.
>>>=20
>>>=20
>>>=20
>>>=20
>>>=20
>=20
> --
> Best Regards,
> Max K.
>=20
>=20
>=20
>=20

--
Best Regards,
Max K.