[
https://issues.apache.org/jira/browse/IGNITE-4358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15729240#comment-15729240
]
Alexei Scherbakov commented on IGNITE-4358:
-------------------------------------------
Rohit,
Thanks for contribution.
I suppose it's better to do null checking before sending a message with empty
runnable and report it to sender.
You also should change the error message accordingly, something like
Trying to execute a null closure. Make sure the closure is not assignable from
classes listed in MarshallerExclusions
> Better error reporting in case of unmarshallable classes.
> ---------------------------------------------------------
>
> Key: IGNITE-4358
> URL: https://issues.apache.org/jira/browse/IGNITE-4358
> Project: Ignite
> Issue Type: Improvement
> Components: compute, messaging, newbie
> Affects Versions: 1.6
> Reporter: Alexei Scherbakov
> Assignee: Rohit Mohta
> Priority: Trivial
> Labels: newbie
> Fix For: 2,0
>
> Attachments: IGNITE-4358-Exceptionlog-05Dec2016.txt,
> IGNITE-4358-GridClosureProcessor-05Dec2016.patch, PureIgniteRunTest.java
>
>
> When trying to execute Thread's derived class implementing IgniteRunnable
> using compute API, it silently serializes to null because Thread
> serialization is prohibited in MarshallerExclusions and throws NPE on
> executing node.
> We need to throw more informative exception for such case.
> Reproducer in the attachment.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
