On Fri, Apr 20, 2018 at 4:50 AM, Anton Vinogradov <[email protected]> wrote:
> P.s. Andrey Kuznetsov, corrected me that we have no warranty that failed
> node able to notify cluster.
>
> But,
>
> try{
> sendDiscoveryMessageWithFail(...);
> } catch(){
> // No-op;
> }
>
> is better than nothing, I think.
>
Agree about the "better than nothing" part, but do not agree about the
"no-op" in the catch block. We should still log the fact that sending of
the failure message failed and provide the exception stack trace if there
is one.
