[
https://issues.apache.org/jira/browse/IGNITE-1272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14903333#comment-14903333
]
Denis Magda commented on IGNITE-1272:
-------------------------------------
Improved the fix for compute engine's class loader in order to recover other
broken tests. Now compute engine is able to reload a class if its version is
changed.
Checked the fix with {{GridPortableObjectsTestSuite}} locally, everything looks
fine.
What is left is to fix the following tests:
- {{GridDeploymentMessageCountSelfTest.testCacheValueDeploymentOnPut()}};
- {{IgniteCacheAbstractExecutionContextTest.testUserClassLoader() fails with
PortableMarshaller enabled}}
They both should failing because of the issue in {{PortableObjectImpl}} I'm
talking above.
> PortableMarshaller: issues when different class loaders are used
> ----------------------------------------------------------------
>
> Key: IGNITE-1272
> URL: https://issues.apache.org/jira/browse/IGNITE-1272
> Project: Ignite
> Issue Type: Bug
> Affects Versions: ignite-1.4
> Reporter: Denis Magda
> Assignee: Denis Magda
> Priority: Blocker
> Fix For: ignite-1.5
>
>
> The reason is that a loader is not passed to required places when needed.
> Reproduced with the following tests:
> - {{IgniteCacheAbstractExecutionContextTest.testUserClassLoader()}} fails
> with PortableMarshaller enabled.
> - {{GridDeploymentMessageCountSelfTest.testCacheValueDeploymentOnPut()}}
> Another issue is when {{PortableContext}} returns {{PortableClassDescriptor}}
> by type id. Returned descriptor has a constructor which already has been
> loaded with another class loader. Fix is not trivial and issue is reproduced
> with {{GridP2PRemoteClassLoadersSelfTest}}
> Look for corresponding TODOs in the code.
> Unmute tests when fixed
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
