Deprecate IgniteCache.query(Query<T>, IgniteClosure<T,R>)

[Vladimir Ozerov]

Igniters,

We have that very strange method, which accepts transformer. It is only
supported for ScanQuery and we do not have any plans to extend it in
future.

Let's just deprecate it and add transformer to ScanQuery directly.

Thoughts?

[Nikolay Izhikov]

+1 from me as I proposed similar design in my implementation of
ContinuousQueryWithTransformer [1]

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


2017-09-11 17:13 GMT+03:00 Vladimir Ozerov <[hidden email]>:

> Igniters,
>
> We have that very strange method, which accepts transformer. It is only
> supported for ScanQuery and we do not have any plans to extend it in
> future.
>
> Let's just deprecate it and add transformer to ScanQuery directly.
>
> Thoughts?
>

--
Nikolay Izhikov
[hidden email]

[dsetrakyan]

On Mon, Sep 11, 2017 at 7:13 AM, Vladimir Ozerov <[hidden email]>
wrote:

> Igniters,
>
> We have that very strange method, which accepts transformer. It is only
> supported for ScanQuery and we do not have any plans to extend it in
> future.
>
> Let's just deprecate it and add transformer to ScanQuery directly.
>
> Thoughts?
>

Vladimir, what is wrong with this method?

[Valentin Kulichenko]

Vova,

I doubt you'll be able to easily add it directly to ScanQuery because of
generics. Do you have an alternative API in mind?

-Val

On Mon, Sep 11, 2017 at 7:32 AM, Dmitriy Setrakyan <[hidden email]>
wrote:

> On Mon, Sep 11, 2017 at 7:13 AM, Vladimir Ozerov <[hidden email]>
> wrote:
>
> > Igniters,
> >
> > We have that very strange method, which accepts transformer. It is only
> > supported for ScanQuery and we do not have any plans to extend it in
> > future.
> >
> > Let's just deprecate it and add transformer to ScanQuery directly.
> >
> > Thoughts?
> >
>
> Vladimir, what is wrong with this method?
>

[Vladimir Ozerov]

Val,

You are right, generics will not allow us to do that. Sad.

On Mon, Sep 11, 2017 at 7:35 PM, Valentin Kulichenko <
[hidden email]> wrote:

> Vova,
>
> I doubt you'll be able to easily add it directly to ScanQuery because of
> generics. Do you have an alternative API in mind?
>
> -Val
>
> On Mon, Sep 11, 2017 at 7:32 AM, Dmitriy Setrakyan <[hidden email]>
> wrote:
>
> > On Mon, Sep 11, 2017 at 7:13 AM, Vladimir Ozerov <[hidden email]>
> > wrote:
> >
> > > Igniters,
> > >
> > > We have that very strange method, which accepts transformer. It is only
> > > supported for ScanQuery and we do not have any plans to extend it in
> > > future.
> > >
> > > Let's just deprecate it and add transformer to ScanQuery directly.
> > >
> > > Thoughts?
> > >
> >
> > Vladimir, what is wrong with this method?
> >
>