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

Maintain deterministic order of CLUSTER SHARDS response #411

Open
wants to merge 3 commits into
base: unstable
Choose a base branch
from

Commits on May 1, 2024

  1. Maintain deterministic order of CLUSTER SHARDS response

    Maintain deterministic order of CLUSTER SHARDS response. Currently we
    don't maintain the shards/masters in sorted fashion and hence we get the
    order of CLUSTER SHARDS response non-deterministic on different nodes.
    Maintain the sorted Masters list of pointers, similar to replicas,
    and get rid of <shards, list<nodes>> dict which is not suitable
    for sorting. Add TOPOLOGY argument to get the deterministic response
    which would remove the replication offset and node health status from
    cluster shards response. Sort the nodes based on the node Id. Use it
    in proc `cluster_config_consistent` for the test coverage and sanity
    purpose.
    
    Signed-off-by: Ram Prasad Voleti <ramvolet@amazon.com>
    Ram Prasad Voleti committed May 1, 2024
    Configuration menu
    Copy the full SHA
    715816a View commit details
    Browse the repository at this point in the history

Commits on May 17, 2024

  1. Remove topology argument and cleanup related code changes

    Remove topology argument and cleanup related code changes.
    
    Signed-off-by: Ram Prasad Voleti <ramvolet@amazon.com>
    Ram Prasad Voleti committed May 17, 2024
    Configuration menu
    Copy the full SHA
    22fc336 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    f9ae110 View commit details
    Browse the repository at this point in the history