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-3942] Remove mvn leftovers from jenkins IOIT job definitions #5812
Conversation
Run seed job |
Run Java MongoDBIO Performance Test |
Run Java JdbcIO Performance Test |
Run Java TextIO Performance Test HDFS |
Run Java AvroIO Performance Test |
R: @iemejia Could you take a quick 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.
LGTM
beam_it_options : common_job_properties.joinPipelineOptions(pipelineOptions), | ||
beam_extra_mvn_properties: '["filesystem=gcs"]', | ||
bigquery_table : testConfiguration.bqTable, | ||
benchmarks : 'beam_integration_benchmark', |
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.
Minor question, is this reformatting automatic? I kind of dislike having everything aligned to colons but it seems this is common in groovy.
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 is automatic.
This PR removes maven leftovers that weren't removed during migration (to migrate seamlessly).
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)