c4emmmm commented 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. What do you think? 
@sjwiesman 

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

Reply via email to