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
[BEAM-9503] Insert missing comma in process worker script. #11123
Conversation
Run Python2 Postcommit |
Run Python3 Postcommit |
Run Python Postcommit |
@ibzib do you know what is the relevant postcommit? Could we run those as well? |
This bug has been around for a while, so apparently this code branch is not being tested. We will have to follow up and investigate why. |
LGTM |
Run Python 2 PostCommit |
Run Python 3.5 PostCommit |
Run Python 3.6 PostCommit |
Run Python 3.7 PostCommit |
Python 3.5 and 3.7 postcommits flaked because of BEAM-8866 |
Run Python 3.5 PostCommit |
Run Python 3.7 PostCommit |
@amaliujia I'm not sure why this bug hasn't been caught before, but this whole code path is broken and it's going to block TFX-on-Flink, so I request a cherry-pick for this PR.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
R: @username
).[BEAM-XXX] Fixes bug in ApproximateQuantiles
, where you replaceBEAM-XXX
with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.CHANGES.md
with noteworthy changes.See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)
Pre-Commit Tests Status (on master branch)
See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.