-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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-4774] Nexmark SQL Postcommits #5947
Conversation
R: @echauchot |
LGTM |
Hit flaps in |
run java precommit |
|
run java precommit |
@apilloud yes there was some side-effect flakiness introduced in ElasticsearchIOTest lately. I need to take a look |
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.
Seems good. I cannot be 100% sure because we can not longer (disabled) run postCommits using phrase triggering. We'll see with next commit to master. Besides, with this PR you'll add numberOfQueriesImplementedInSQL x 3Runners x 2Modes (except spark streaming) tables in nexmark big query dataset.
@apilloud can you squash the 2 commits into one and rename it with the ticket number? |
I've squashed and renamed the commit. |
This adds the Nexmark SQL jobs to the post commits.
Follow this checklist to help us incorporate your contribution quickly and easily:
[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.It will help us expedite review of your Pull Request if you tag someone (e.g.
@username
) to look at it.Post-Commit Tests Status (on master branch)