-
Notifications
You must be signed in to change notification settings - Fork 302
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add flink-ml 2.0.0 #493
Add flink-ml 2.0.0 #493
Conversation
b2f0246
to
0b2c050
Compare
downloads.md
Outdated
</dependency> | ||
``` | ||
|
||
flink-ml-uber_2.12 provides all the dependencies needed to use Flink ML. Alternatively, advanced users could use the following artifacts to import a minimal set of Flink ML dependencies for their target use-cases: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that flink-ml-uber_2.12 only includes flink-ml-core
and flink-ml-iteration
[1]. Do you mean to include all other dependencies in flink-ml?
[1] https://github.com/apache/flink-ml/blob/master/flink-ml-uber/pom.xml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @zhipeng93 I think you are right, I removed dependency of flink-ml-uber
currently and use directly the direct dependencies . We could re-enabled the uber
jar in the next version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for creating this PR @gaoyunhaii. I think this PR is missing the rebuild commit.
85d530e
to
8b1ff41
Compare
Very thanks @tillrohrmann for the review! I add the rebuild commit now~ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we also update ./content/zh/downloads.html
? Thanks!
@lindong28 I added the translation~ |
@gaoyunhaii Thanks for the update! I have checked both |
This PR updates the documents to include the downloads information for the upcoming flink-ml 2.0.0