Reuse of `br` in the do-loop looks kind of surprising. I think the auto-closing of `soMaxConnIsr` closes `br` on line 377 (`}`) so when you loop back around the `br.close()` on line 369 is a no-op... but I'm not 100% positive. I'd use a new var that only exists within the inner-try.
[ Full content available at: https://github.com/apache/geode/pull/3036 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org
