gaoyunhaii commented on a change in pull request #25:
URL: https://github.com/apache/flink-ml/pull/25#discussion_r746646216



##########
File path: 
flink-ml-iteration/src/main/java/org/apache/flink/iteration/operator/perround/AbstractPerRoundWrapperOperator.java
##########
@@ -223,7 +246,44 @@ public void initializeState(StreamTaskStateInitializer 
streamTaskStateManager)
 
     @Override
     public void initializeState(StateInitializationContext context) throws 
Exception {
-        // Do thing for now since we do not have states.
+        parallelismState =

Review comment:
       I also agree we'd better always call super method, but when I add the 
code it suddenly comes to the that the operator does not extend the 
`AbstractStreamOperator`, thus in this case we do not have the super method to 
call~




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to