c4emmmm commented on issue #8526: [FLINK-12597][ml] Remove the legacy flink-libraries/flink-ml URL: https://github.com/apache/flink/pull/8526#issuecomment-507495317 > @c4emmmm Agreed, fell free to pull in my commit from #8827 or I can reopen the PR. As part of FLIP-42 we are restructuring the docs and decided to fully remove the old FlinkML documentation. If users are still using this library from an old version, they can also use an old version of the docs. I would be opposed to adding an "under construction page", I find that more confusing. Thanks. I also oppose that with deeper thought so I mark it with delete line. Since the modification is almost done in #8827, I think it's better to reopen the PR and merge it. The only docs that mentioning FlinkML are docs/internals/components.md and components.zh.md. You can find the keyword "FlinkML" and decide whether to remove them. @sjwiesman , would you please spend a little time doing this if @shaoxuan-wang and @zentol agree we merge this and #8827 separately? And what do you think? @shaoxuan-wang @zentol
---------------------------------------------------------------- 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
