becketqin commented on issue #9184: [FLINK-13339][ml] Add an implementation of 
pipeline's api
URL: https://github.com/apache/flink/pull/9184#issuecomment-541362792
 
 
   @xuyang1706 Thanks for updating the patch. the patch LGTM overall. I created 
a PR with some minor improvements against your branch. Can you check if that 
makes sense? If so, feel free to merge it to your branch.
   
   @walterddr Do you also want to take another look? I am thinking of merging 
the patch either on Sunday or Monday. Thanks.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to