c4emmmm commented on issue #8402: [FLINK-12473][ml] Add the interface of ML pipeline and ML lib URL: https://github.com/apache/flink/pull/8402#issuecomment-495538481 @sunjincheng121 Thanks for your review and the reminder to add flink-ml into flink-dist, I'll do it in later PRs. And I'll take the suggestions in javaDocs. As for the module name, I think it's correct that we should avoid confusing users with using same name indicating completely different modules even in different versions. But I'm also afraid that 'flink-ml-parent' is not a proper name. I'm not sure but maybe we should have only one 'parent' in flink. I'd like to ask more ideas before I change the module name again.
---------------------------------------------------------------- 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
