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)