You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Bill Burcham (Jira)" <ji...@apache.org> on 2020/10/07 17:12:00 UTC

[jira] [Commented] (GEODE-7696) Remove unecessary catch block from GMSHealthMonitor.java

    [ https://issues.apache.org/jira/browse/GEODE-7696?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17209699#comment-17209699 ] 

Bill Burcham commented on GEODE-7696:
-------------------------------------

When the ticket says "we can let both bubble up" is the ticket suggesting we remove the {{catch}} block for {{IllegalStateException}}?

If so, what is the justification for that? Why is it ok to stop catching {{IllegalStateException}}?

> Remove unecessary catch block from GMSHealthMonitor.java
> --------------------------------------------------------
>
>                 Key: GEODE-7696
>                 URL: https://issues.apache.org/jira/browse/GEODE-7696
>             Project: Geode
>          Issue Type: Improvement
>          Components: membership
>            Reporter: Ernest Burghardt
>            Priority: Major
>
> former:
> {code:java}
> } catch (IllegalStateException \| GemFireConfigException e) {
> {code}
> current:
> {code:java}
> } catch (IllegalStateException e) {
> {code}
> we can let both bubble up...



--
This message was sent by Atlassian Jira
(v8.3.4#803005)