Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[STORM-3017] Refactor pacemaker client exception handling #2621

Merged
merged 1 commit into from Apr 23, 2018
Merged

[STORM-3017] Refactor pacemaker client exception handling #2621

merged 1 commit into from Apr 23, 2018

Conversation

Ethanlm
Copy link
Contributor

@Ethanlm Ethanlm commented Apr 3, 2018

https://issues.apache.org/jira/browse/STORM-3017

Better exception handling for pacemaker client

@Ethanlm
Copy link
Contributor Author

Ethanlm commented Apr 18, 2018

@revans2 @srdo Could you please take a look if you get a chance? Thanks

@srdo
Copy link
Contributor

srdo commented Apr 19, 2018

+1, thanks for fixing this.

revans2
revans2 approved these changes Apr 23, 2018
View reviewed changes
Copy link
Contributor

@revans2 revans2 left a comment

Choose a reason for hiding this comment

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

+1

@asfgit asfgit merged commit 8b91730 into apache:master Apr 23, 2018
asfgit pushed a commit that referenced this pull request Apr 23, 2018
Merge branch 'STORM-3017' of https://github.com/Ethanlm/storm into ST…
7673253 
…ORM-3017

STORM-3017: Refactor pacemaker client exception handling

This closes #2621
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@revans2 revans2 revans2 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@Ethanlm @srdo @revans2 @asfgit