[
https://issues.apache.org/jira/browse/STORM-468?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15083120#comment-15083120
]
ASF GitHub Bot commented on STORM-468:
--------------------------------------
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/477#issuecomment-169014532
+1 for the concept. The formatting of all of the code is off. Looks like
they are using tabs, not spaces. I would also like to see @HeartSaVioR's
comment addressed.
There also appear to be merge conflicts now. @jangie sorry this has
lingered so very long. This is a great concept. Is there any hope that you
can address these comments?
> java.io.NotSerializableException should be explained
> ----------------------------------------------------
>
> Key: STORM-468
> URL: https://issues.apache.org/jira/browse/STORM-468
> Project: Apache Storm
> Issue Type: Bug
> Components: storm-core
> Affects Versions: 0.9.2-incubating
> Environment: Any
> Reporter: Jason Kania
> Priority: Minor
> Labels: newbie
>
> The occurrence of the NotSerializableException and how to avoid it should be
> better documented and the error from the code should be expanded to include
> some human readable message because it took a lot of searching to find out
> that the spouts and bolts need to create their variables in the prepare
> method in order to avoid this problem.
> The error text output could state that this is how to solve the problem.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
