Skip to content

[BEAM-4618] Fix errors in BeamModulePlugin perf test section#5732

Merged
kennknowles merged 1 commit intoapache:masterfrom
kennknowles:perf-tests
Jun 22, 2018
Merged

[BEAM-4618] Fix errors in BeamModulePlugin perf test section#5732
kennknowles merged 1 commit intoapache:masterfrom
kennknowles:perf-tests

Conversation

@kennknowles
Copy link
Copy Markdown
Member

The BeamModulePlugin section on perf tests was not migrated adequately from build_rules.gradle


Follow this checklist to help us incorporate your contribution quickly and easily:

  • Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

@kennknowles
Copy link
Copy Markdown
Member Author

R: @lgajowy

FYI this is where the code moved, and what I had to do to make it work in Gradle plugin code. It is very slightly different, in boring ways, than the Gradle DSL.

@kennknowles
Copy link
Copy Markdown
Member Author

Run Java AvroIO Performance Test

@kennknowles
Copy link
Copy Markdown
Member Author

Run Java TextIO Performance Test HDFS

@lgajowy
Copy link
Copy Markdown
Contributor

lgajowy commented Jun 22, 2018

LGTM. Thanks for the fix!

@kennknowles kennknowles merged commit 7bf5966 into apache:master Jun 22, 2018
@kennknowles kennknowles deleted the perf-tests branch July 3, 2018 21:56
charlesccychen pushed a commit to charlesccychen/beam that referenced this pull request Jul 24, 2018
charlesccychen pushed a commit to charlesccychen/beam that referenced this pull request Jul 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants