You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Vladimir Ozerov <vo...@gridgain.com> on 2015/02/09 10:50:37 UTC

Cache iterator serialization support.

Hi,

Previously GridGain's cache iterator was serializable. Now as we moved to
Ignite iterator is not serializable anymore and several old unit tests fail
because of it.

As java.util.Iterator is not serializable, I just removed failing tests
without any code fixes.

Please let me know if anyone has objections.

Vladimir.

Re: Cache iterator serialization support.

Posted by Valentin Kulichenko <va...@gmail.com>.
+1

On Mon, Feb 9, 2015 at 8:41 AM, Dmitriy Setrakyan <ds...@apache.org>
wrote:

> I also don't think iterators should be serializable.
>
> On Mon, Feb 9, 2015 at 7:49 AM, Yakov Zhdanov <yz...@gridgain.com>
> wrote:
>
> > I vote for NOT supporting serialization for iterators.
> >
> > --
> > Yakov Zhdanov, Director R&D
> > *GridGain Systems*
> > www.gridgain.com
> >
> > 2015-02-09 12:50 GMT+03:00 Vladimir Ozerov <vo...@gridgain.com>:
> >
> > > Hi,
> > >
> > > Previously GridGain's cache iterator was serializable. Now as we moved
> to
> > > Ignite iterator is not serializable anymore and several old unit tests
> > fail
> > > because of it.
> > >
> > > As java.util.Iterator is not serializable, I just removed failing tests
> > > without any code fixes.
> > >
> > > Please let me know if anyone has objections.
> > >
> > > Vladimir.
> > >
> >
>

Re: Cache iterator serialization support.

Posted by Dmitriy Setrakyan <ds...@apache.org>.
I also don't think iterators should be serializable.

On Mon, Feb 9, 2015 at 7:49 AM, Yakov Zhdanov <yz...@gridgain.com> wrote:

> I vote for NOT supporting serialization for iterators.
>
> --
> Yakov Zhdanov, Director R&D
> *GridGain Systems*
> www.gridgain.com
>
> 2015-02-09 12:50 GMT+03:00 Vladimir Ozerov <vo...@gridgain.com>:
>
> > Hi,
> >
> > Previously GridGain's cache iterator was serializable. Now as we moved to
> > Ignite iterator is not serializable anymore and several old unit tests
> fail
> > because of it.
> >
> > As java.util.Iterator is not serializable, I just removed failing tests
> > without any code fixes.
> >
> > Please let me know if anyone has objections.
> >
> > Vladimir.
> >
>

Re: Cache iterator serialization support.

Posted by Yakov Zhdanov <yz...@gridgain.com>.
I vote for NOT supporting serialization for iterators.

--
Yakov Zhdanov, Director R&D
*GridGain Systems*
www.gridgain.com

2015-02-09 12:50 GMT+03:00 Vladimir Ozerov <vo...@gridgain.com>:

> Hi,
>
> Previously GridGain's cache iterator was serializable. Now as we moved to
> Ignite iterator is not serializable anymore and several old unit tests fail
> because of it.
>
> As java.util.Iterator is not serializable, I just removed failing tests
> without any code fixes.
>
> Please let me know if anyone has objections.
>
> Vladimir.
>