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.

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

PAYARA-2781 Exception thrown in logs when shutting down Payara #2799

Merged
merged 5 commits into from May 31, 2018

Conversation

Cousjava
Copy link
Contributor

No description provided.

@Cousjava Cousjava added this to the Payara 5.182 milestone May 30, 2018
@Cousjava
Copy link
Contributor Author

Jenkins test please

Copy link
Contributor

@lprimak lprimak left a comment

Choose a reason for hiding this comment

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

Swallowing exceptions. Perhaps it could be logged as FINE or something else

@lprimak
Copy link
Contributor

lprimak commented May 30, 2018

Feel free to dismiss my review if I am way off base here

@payara-ci
Copy link
Contributor

Quick build and test failed!

@Cousjava
Copy link
Contributor Author

Jenkins test please

@Cousjava Cousjava dismissed lprimak’s stale review May 31, 2018 08:16

NPE is thrown if JMX isn't started, there aleady is a null check for one point but this is simpler then lots of null checks

@payara-ci
Copy link
Contributor

Quick build and test failed!

@Cousjava
Copy link
Contributor Author

Jenkins test please

@payara-ci
Copy link
Contributor

Quick build and test failed!

@Pandrex247
Copy link
Member

Jenkins test please

@payara-ci
Copy link
Contributor

Quick build and test passed!

@Cousjava Cousjava merged commit ca5626d into payara:master May 31, 2018
Cousjava added a commit to Cousjava/Payara that referenced this pull request May 31, 2018
…a#2799)

* PAYARA-2781 Exception thrown in logs when shutting down Payara
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.

None yet

4 participants