ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Yakov Zhdanov <yzhda...@apache.org>
Subject Re: Potential OOM while iterating over query cursor. Review needed.
Date Wed, 18 Jul 2018 12:14:51 GMT
Yes! Just deprecate getAll() and change default keepAll for scans to false.

--Yakov

2018-07-18 13:39 GMT+03:00 Alexey Goncharuk <alexey.goncharuk@gmail.com>:

> Folks,
>
> There is no need to add getNext() method because the object we are
> discussing is already an iterator. Then, to summarize the solution, we are
> going to deprecate getAll() method and set keepAll flag to false for scan
> query.
>
> Agree?
>
> пн, 16 июл. 2018 г. в 23:40, Dmitriy Setrakyan <dsetrakyan@apache.org>:
>
> > On Mon, Jul 16, 2018 at 5:42 PM, Yakov Zhdanov <yzhdanov@apache.org>
> > wrote:
> >
> > > Dmitry, let's have only getNext() same as jdbc. All other shortcuts
> seem
> > to
> > > overload API without adding much value.
> > >
> >
> > Agree. Do you mind creating a ticket?
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message