Skip to content
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

CCS: don't proxy requests for already connected node #31273

Merged
merged 4 commits into from
Jun 13, 2018

Commits on Jun 11, 2018

  1. CCS: don't proxy requests for already connected node

    Cross-cluster search selects a subset of nodes for each remote cluster
    and sends requests only to them, which will act as a proxy and properly
    redirect such requests to the target nodes that hold the relevant data.
    What happens today is that every time we send a request to a remote
    cluster, it will be sent to the next node in the proxy list
    (in round-robin fashion), regardless of whether the target node is
    already amongst the ones that we are connected to. In case for instance
    we need to send a shard search request to a data node that's also one of
    the selected proxy nodes, we may end up sending the request to it
    through one of the other proxy nodes.
    This commit optimizes this case to make sure that whenever we are
    already connected to a remote node, we will send a direct request rather
    than using the next proxy node.
    There is a side-effect to this, which is that round-robin will be a bit
    unbalanced as the data nodes that are also selected as proxies will
    receive more requests.
    javanna committed Jun 11, 2018
    Configuration menu
    Copy the full SHA
    23b3a40 View commit details
    Browse the repository at this point in the history

Commits on Jun 12, 2018

  1. Configuration menu
    Copy the full SHA
    43c0d5e View commit details
    Browse the repository at this point in the history

Commits on Jun 13, 2018

  1. address review comments

    javanna committed Jun 13, 2018
    Configuration menu
    Copy the full SHA
    8979a5b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    193285a View commit details
    Browse the repository at this point in the history