-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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 portable client configurations #31188
flink portable client configurations #31188
Conversation
Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment |
0af945d
to
d5957f8
Compare
Assigning reviewers. If you would like to opt out of this review, comment R: @damondouglas added as fallback since no labels match configuration Available commands:
The PR bot will only process comments in the main thread (not review comments). |
Hi @AyWa and @damondouglas I just wonder when this feature may be available. I need to increase the job submission timeout as my python beam pipeline keeps failing due to the 30 seconds limit. Or can I have a temporary jar file (beam-runners-flink-1.17-job-server-2.56.0.jar) with this feature until it gets included? Cheers, |
Hey, It is possible to build the jar, I am a bit busy today, but I will try to come back to you soon. A temporary option is to increase cpu allocation of the job manager. (It is how I temporary fixed it in my k8s cluster) |
Hi @AyWa Thanks for your reply. I'm trying to deploy a python pipeline to a minikube cluster and I don't have much luck by adjusting cpu allocation. It'll be great if the jar file is shared or I'm informed of how to build it. Cheers, |
Not sure it is the best way, but this is how I am doing: |
Reminder, please take a look at this pr: @damondouglas |
Assigning new set of reviewers because Pr has gone too long without review. If you would like to opt out of this review, comment R: @damondouglas added as fallback since no labels match configuration Available commands:
|
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.
LGTM. Thank you @AyWa for your contribution!
Descriptions
Previously made an attempt to solve that in https://github.com/apache/beam/pull/28068/files, but at that time, I didn't have time to follow up.
I will try to contribute better to beam (especially for flink runner).
In this PR, we make
FlinkPortableClientEntryPoint
accept parameters thatJobServerDriver
can handle.