Skip to content

TransportClusterStateAction should not (always) sent the full state over the wire #86888

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
original-brownbear opened this issue May 18, 2022 · 2 comments
Labels
>bug :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination.

Comments

@original-brownbear
Copy link
Member

Currently, TransportClusterStateAction sends a ClusterStateResponse that contains the full cluster state serialized and uncompressed. This kind of response should never be necessary because the node sending the ClusterStateRequest should already have an up-to-date cluster state available locally.
We should be able to find a better scheme for requesting the current cluster state for usages like RestClusterStateAction that does not involve serializing a full state over the wire (and as a result of that also results in instantiating a duplicate instance to the node's local cluster state on heap) by reusing the node's local state and only verifying that it is up-to-date with the master node.

relates #77466

@original-brownbear original-brownbear added >enhancement :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. labels May 18, 2022
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label May 18, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination.
Projects
None yet
Development

No branches or pull requests

4 participants
@original-brownbear @elasticmachine @henningandersen and others