Skip to content
This repository has been archived by the owner on Jan 8, 2019. It is now read-only.

Forked Smoke-Tests #1241

Merged
merged 1 commit into from
Jul 18, 2018
Merged

Forked Smoke-Tests #1241

merged 1 commit into from
Jul 18, 2018

Conversation

IshChu
Copy link

@IshChu IshChu commented Jul 16, 2018

Currently tests are serialized so if one fails, all following tests won't even be run. We want to make sure every test is able to be run to test every service. The best way to do this is using fork + join + decision. If one of these fails, it won't goto a fail path and end oozie execution because all fail/success paths goto the join. Once all tests are done, a decision is used to check if lastErrorNode() was populated or not which would indicate whether or not something failed.

Also removed a duplicate line from job properties.

removed duplicate line from properties
Copy link
Member

@vt0r vt0r left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@vt0r vt0r removed the Enhancement label Jul 17, 2018
@bijugs
Copy link
Contributor

bijugs commented Jul 18, 2018

LGTM

1 similar comment
@macmaster
Copy link
Contributor

LGTM

@vt0r vt0r merged commit 7d657b5 into bloomberg:master Jul 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants