jiangxin369 commented on code in PR #180:
URL: https://github.com/apache/flink-ml/pull/180#discussion_r1031178423


##########
flink-ml-python/pyflink/ml/lib/tests/test_ml_lib_completeness.py:
##########
@@ -81,6 +83,9 @@ def module_name(self):
     def module(self):
         pass
 
+    def exclude_java_stage(self):

Review Comment:
   I don't think we need to add a comment to a test method that is not 
difficult to understand, but we can leave the description here for developers' 
reference.
   
   For those java stages that have no corresponding python implementation, 
developers can skip testing them by overriding this function and listing them 
in the format `package.class`, e.g. `anovatest.ANOVATest`.
   



-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to