[GitHub] [flink] c4emmmm edited a comment on issue #8526: [FLINK-12597][ml] Remove the legacy flink-libraries/flink-ml

[GitBox <gi...@apache.org>]

c4emmmm edited a comment on issue #8526: [FLINK-12597][ml] Remove the legacy flink-libraries/flink-ml
URL: https://github.com/apache/flink/pull/8526#issuecomment-507152953
 
 
   > Very good point, @c4emmmm , can you please consider this suggestion and make some actions on the existing docs. Maybe you can just include the changes made in [FLINK-12928](https://github.com/apache/flink/pull/8827). We also need to open a JIRA to update flink web later.
   
   Now that @sjwiesman has already proposed the PR, and it seems to be what we need to do, I suppose that we can simply reopen and accept that PR as soon as we accept this PR.
   There's a problem that in [FLINK-12928](https://github.com/apache/flink/pull/8827) the descriptions and links of legacy FlinkML are only redirected rather than removed, which may confuse users. 
   I can suggest some changes to the PR before it is merged, or take over it if @sjwiesman currently has no time to do this. Another option is to add a "New FlinkML Under Construction" page, and redirect remaining FlinkML links to this page. I think that's enough meanwhile more simple and quick. 
   
   What do you think? @sjwiesman @shaoxuan-wang 

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


With regards,
Apache Git Services