yunfengzhou-hub commented on pull request #60: URL: https://github.com/apache/flink-ml/pull/60#issuecomment-1046377495
Thanks for the PR! I think it's a crucial feature for Flink ML. I suppose this PR also wants to solve the problem raised in [this email](https://lists.apache.org/thread/2cs3g872lbyzqyjh4xjkn52fhko4mo0y). In this email an exception was thrown when an unbounded stream was fed to an Estimator, but the test cases introduced by this PR have only covered the bounded situations. It might be better if we could add test cases that corresponds to the conditions described in the email. -- 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