### Description

The `sendRequest` method is asynchronous and notifies a 
`TransportResponseHandler` when it completes the excecution
(successfully or unsuccessfully). 

When opening the connection we only catch `NodeNotConnectedException`
( https://github.com/elastic/elasticsearch/blob/main/server/src/main/java/org/elasticsearch/transport/TransportService.java#L719 ) which is currently fine as no implementations
throw other type of exception however, it'd be best to catch `Exception` here
so the `handler` gets notified of potential exceptions (as opposed to the caller
having to `try/catch` the call to `sendRequest`)
server/src/main/java/org/elasticsearch/cluster/coordination/PublicationTransportHandler.java
assert false;
                listener.onFailure(new IllegalStateException("serialized cluster state released before transmission"));
                return;
            }
            try {
                transportService.sendChildRequest(
                    destination,
                    PUBLISH_STATE_ACTION_NAME,
                    new BytesTransportRequest(bytes, destination.getVersion()),
                    task,
                    STATE_REQUEST_OPTIONS,
                    new ActionListenerResponseHandler<>(
                        ActionListener.runAfter(listener, bytes::decRef),
                        PublishWithJoinResponse::new,
                        ThreadPool.Names.CLUSTER_COORDINATION
                    )
                );
            } catch (Exception e) {
                assert false : e;
                logger.warn(() -> format("error sending cluster state to %s", destination), e);
                listener.onFailure(e);
            }
        }
        @Override
        protected void closeInternal() {
            serializedDiffs.values().forEach(Releasables::closeExpectNoException);
            serializedStates.values().forEach(Releasables::closeExpectNoException);
        }
    }
    private static class SerializationStatsTracker {