xuyang1706 commented on issue #9184: [FLINK-13339][ml] Add an implementation of 
pipeline's api
URL: https://github.com/apache/flink/pull/9184#issuecomment-541380802
 
 
   > Thanks for the contribution @xuyang1706 and the reviews @becketqin . 
Overall it looks good to me. I only left a minor comment. please kindly take a 
look.
   > +1 to merge to unblock future developments.
   
   Thanks @walterddr. Yes, this API is rarely used, but in some cases, if user 
want to release no-used MLEnviroment in the process, this is only API could be 
called. Thus, I prefer to keep it.

----------------------------------------------------------------
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