hequn8128 edited a comment on issue #11220: [FLINK-16249][python][ml] Add interfaces for Params, ParamInfo and WithParams URL: https://github.com/apache/flink/pull/11220#issuecomment-591807495 > @hequn8128 Thanks for the PR. > > 1. I noticed that there are two modules "ml" and "mllib" introduced. I'm wondering if one module such as ml is enough? @dianfu This sounds like a good idea. We can add api and lib under the `pyflink.ml` package, i.e., `pyflink.ml.api` and `pyflink.ml.lib`. The package directly under the pyflink should be an entire module, somehow consistent with `pyflink.table`. @becketqin @walterddr What's your opinion on this. If we reach a consensus on this, we may need an extra vote since it is different from the FLIP wiki page.
---------------------------------------------------------------- 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
