Skip to content

[BEAM-5683] Print command logs on failure#6646

Merged
pabloem merged 1 commit into
apache:masterfrom
angoenka:dataflow_vr_pip_log
Oct 12, 2018
Merged

[BEAM-5683] Print command logs on failure#6646
pabloem merged 1 commit into
apache:masterfrom
angoenka:dataflow_vr_pip_log

Conversation

@angoenka
Copy link
Copy Markdown
Contributor

Please add a meaningful description for your change here


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.

Post-Commit Tests Status (on master branch)

Lang SDK Apex Dataflow Flink Gearpump Samza Spark
Go Build Status --- --- --- --- --- ---
Java Build Status Build Status Build Status Build Status Build Status Build Status Build Status
Python Build Status --- Build Status
Build Status
Build Status --- --- ---

@angoenka angoenka force-pushed the dataflow_vr_pip_log branch from 91017be to 9477e53 Compare October 10, 2018 23:48
@angoenka
Copy link
Copy Markdown
Contributor Author

R: @pabloem

@pabloem
Copy link
Copy Markdown
Member

pabloem commented Oct 11, 2018

Hmmm the failure seems related to the change. I am not sure why it's happening though...

@angoenka angoenka force-pushed the dataflow_vr_pip_log branch from 9477e53 to 8124cfd Compare October 11, 2018 20:18
@angoenka
Copy link
Copy Markdown
Contributor Author

Fixed the test case.

@pabloem
Copy link
Copy Markdown
Member

pabloem commented Oct 12, 2018

Ah thanks Ankur! lgtm

@pabloem pabloem merged commit ef79e9b into apache:master Oct 12, 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