xuyang1706 commented on issue #9184: [FLINK-13339][ml] Add an implementation of 
pipeline's api
URL: https://github.com/apache/flink/pull/9184#issuecomment-535402676
 
 
   > Thanks for the follow up @xuyang1706 . Overall the changes looks good to 
me. I left some minor comments. Also I notices that some large trunk of code 
was missing tests? were we going to support individual tests in the future, or 
they are part of the `MLSessionTest`?
   
   Thanks @walterddr , I refactored the package paths, added some test cases 
and refined the JavaDoc with examples. 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to