Ignite components deserialization

[Valentin Kulichenko]

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

[dsetrakyan]

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 <
[hidden email]> 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
>

[Vladimir Ershov]

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 <
[hidden email]> 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
>