Skip to content

Comments

[BEAM-2015] Remove shared profile in runners/pom.xml and fix Dataflow ValidatesRunner PostCommit#2592

Closed
lukecwik wants to merge 1 commit intoapache:masterfrom
lukecwik:dataflow_post_commit
Closed

[BEAM-2015] Remove shared profile in runners/pom.xml and fix Dataflow ValidatesRunner PostCommit#2592
lukecwik wants to merge 1 commit intoapache:masterfrom
lukecwik:dataflow_post_commit

Conversation

@lukecwik
Copy link
Member

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

  • Make sure the PR title is formatted like:
    [BEAM-<Jira issue #>] Description of pull request
  • Make sure tests pass via mvn clean verify. (Even better, enable
    Travis-CI on your fork and ensure the whole test matrix passes).
  • Replace <Jira issue #> in the title with the actual Jira issue
    number, if there is one.
  • If this contribution is large, please file an Apache
    Individual Contributor License Agreement.

Diff'd effective poms to compare changes and find all inherited properties in sub runner modules.

I was limited by the seed job failing to make some other minor changes in clean-up.

@lukecwik lukecwik force-pushed the dataflow_post_commit branch from ece72d4 to 48193e6 Compare April 19, 2017 16:21
@lukecwik
Copy link
Member Author

@dhalperi
Copy link
Contributor

Run Dataflow ValidatesRunner

@dhalperi
Copy link
Contributor

Run Spark ValidatesRunner

@dhalperi
Copy link
Contributor

Run Apex ValidatesRunner

@dhalperi
Copy link
Contributor

Run Flink ValidatesRunner

@dhalperi
Copy link
Contributor

Run Direct ValidatesRunner

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 70.245% when pulling 48193e6 on lukecwik:dataflow_post_commit into 8319369 on apache:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 70.134% when pulling 48193e6 on lukecwik:dataflow_post_commit into 8319369 on apache:master.

@dhalperi
Copy link
Contributor

dhalperi commented Apr 19, 2017

LGTM!

@dhalperi
Copy link
Contributor

And thanks!

@asfgit asfgit closed this in 19ae877 Apr 19, 2017
Copy link
Member

@davorbonaci davorbonaci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lukecwik lukecwik deleted the dataflow_post_commit branch September 26, 2017 22:13
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.

4 participants