You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@ignite.apache.org by Valentin Kulichenko <va...@gmail.com> on 2016/02/13 00:30:20 UTC

Ignite components deserialization

Folks,

I reopened the ticket where we improved the serialization of Ignite
components [1].

>From what I can see, the fix was made for IgniteKernal, but not for other
classes like ClusterGroupAdapter, GridKernalContextImpl and others. What is
the reason for this?

Vladimir Ershov, it looks like you were working on this, can you please
respond?

[1] https://issues.apache.org/jira/browse/IGNITE-10

-Val

Re: Ignite components deserialization

Posted by Dmitriy Setrakyan <ds...@apache.org>.

I have also looked at the ticket, and I do not see if the review was done
there. Can I please ask all the committers to make sure to add review
comments to the tickets? Otherwise, it is impossible to understand the
history of the fix.

Thanks,
D.

On Fri, Feb 12, 2016 at 3:30 PM, Valentin Kulichenko <
valentin.kulichenko@gmail.com> wrote:

> Folks,
>
> I reopened the ticket where we improved the serialization of Ignite
> components [1].
>
> From what I can see, the fix was made for IgniteKernal, but not for other
> classes like ClusterGroupAdapter, GridKernalContextImpl and others. What is
> the reason for this?
>
> Vladimir Ershov, it looks like you were working on this, can you please
> respond?
>
> [1] https://issues.apache.org/jira/browse/IGNITE-10
>
> -Val
>

Re: Ignite components deserialization

Posted by Vladimir Ershov <ve...@gridgain.com>.

Yes, thanks, already working on it.

The reason was: Ignite experts decided, that there should be no case for
Ignite to be deserialized outside of IgniteThread.

On Sat, Feb 13, 2016 at 2:30 AM, Valentin Kulichenko <
valentin.kulichenko@gmail.com> wrote:

> Folks,
>
> I reopened the ticket where we improved the serialization of Ignite
> components [1].
>
> From what I can see, the fix was made for IgniteKernal, but not for other
> classes like ClusterGroupAdapter, GridKernalContextImpl and others. What is
> the reason for this?
>
> Vladimir Ershov, it looks like you were working on this, can you please
> respond?
>
> [1] https://issues.apache.org/jira/browse/IGNITE-10
>
> -Val
>