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
[FLINK-17422] Create user document for the external resource framewor… #12538
Conversation
Thanks a lot for your contribution to the Apache Flink project. I'm the @flinkbot. I help the community Automated ChecksLast check on commit 6676085 (Tue Jun 09 04:08:27 UTC 2020) ✅no warnings Mention the bot in a comment to re-run the automated checks. Review Progress
Please see the Pull Request Review Guide for a full explanation of the review process. The Bot is tracking the review progress through labels. Labels are applied according to the order of the review items. For consensus, approval by a Flink committer of PMC member is required Bot commandsThe @flinkbot bot supports the following commands:
|
0d89db3
to
9f998ef
Compare
…mework and the GPU plugin
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 @KarmaGYZ. I had few minor comments. Please ping me once you have them resolved. Once they are resolved, I will merge this PR.
|
||
# Existing supported external resource plugins | ||
|
||
Currently, Flink supports GPUs as external resources. |
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.
Maybe add a link to https://github.com/apache/flink/blob/master/flink-examples/flink-examples-streaming/src/main/java/org/apache/flink/streaming/examples/gpu/MatrixVectorMul.java as an example how to use the GPUs.
…rce framework and the GPU plugin
Thanks for the review @tillrohrmann ! I've updated the PR. |
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 addressing my comments @KarmaGYZ. LGTM. Merging this PR now.
…mework and the GPU plugin This closes #12538.
…mework and the GPU plugin This closes apache#12538.
…k and the GPU plugin
What is the purpose of the change
Create user document for the external resource framework and the GPU plugin
Brief change log
Create user document for the external resource framework and the GPU plugin
Verifying this change
Manually build the user doc.
Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: noDocumentation