[
https://issues.apache.org/jira/browse/STORM-1415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15085008#comment-15085008
]
ASF GitHub Bot commented on STORM-1415:
---------------------------------------
Github user vesense commented on a diff in the pull request:
https://github.com/apache/storm/pull/973#discussion_r48929312
--- Diff:
storm-core/src/jvm/storm/trident/state/map/MapReducerAggStateUpdater.java ---
@@ -34,39 +34,38 @@
import storm.trident.tuple.TridentTupleView.ProjectionFactory;
public class MapReducerAggStateUpdater implements StateUpdater<MapState> {
+ //ANY CHANGE TO THIS CODE MUST BE SERIALIZABLE COMPATIBLE OR THERE
WILL BE PROBLEMS
+ private static final long serialVersionUID = 4382969836839584102L;
+
ReducerAggregator _agg;
Fields _groupFields;
Fields _inputFields;
ProjectionFactory _groupFactory;
ProjectionFactory _inputFactory;
--- End diff --
Done.
> Some improvements for trident map StateUpdater
> ----------------------------------------------
>
> Key: STORM-1415
> URL: https://issues.apache.org/jira/browse/STORM-1415
> Project: Apache Storm
> Issue Type: Improvement
> Components: storm-core
> Reporter: Xin Wang
> Assignee: Xin Wang
> Priority: Minor
>
> Changes are the following:
> 1.Add a generated serialVersionUID
> 2.Remove unused variables 'groups' & 'values'
> 3.Add <> in order to pass compiler type check
> 4.Fix newline and whitespaces
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
