You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/01/20 05:05:13 UTC

[GitHub] [flink-ml] lindong28 commented on pull request #54: [FLINK-25552] Add Estimator and Transformer for MinMaxScaler in FlinkML

lindong28 commented on pull request #54:
URL: https://github.com/apache/flink-ml/pull/54#issuecomment-1017123834


   Thanks for the PR.
   
   Before we continue to more and more algorithm, what's our plan to making sure these algorithm's implementation could meet our performance target? I am concerned that if we add more algorithms before we have a benchmark plan, we might end up refactoring added algorithms in a unnecessary manner, which is probably not a good long term solution.


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

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org