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

[Backport-2.x] Cluster State Update Optimization (#7853) #8644

Conversation

sandeshkr419
Copy link
Contributor

@sandeshkr419 sandeshkr419 commented Jul 11, 2023

  • Cluster State Update Optimization - Optimize Metadata build() to skip redundant computations of indicesLookup as part of ClusterState build

Signed-off-by: Sandesh Kumar sandeshkr419@gmail.com


Signed-off-by: Sandesh Kumar sandeshkr419@gmail.com
(cherry picked from commit cb0d13b)

Description

Backporting #7853 to 2.x branch

Related Issues

Resolves #7002

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…7853)

* Cluster State Update Optimization - Optimize Metadata build() to skip redundant computations of indicesLookup as part of ClusterState build

Signed-off-by: Sandesh Kumar <sandeshkr419@gmail.com>

---------

Signed-off-by: Sandesh Kumar <sandeshkr419@gmail.com>
(cherry picked from commit cb0d13b)
@sandeshkr419 sandeshkr419 changed the title [Optimization] Cluster State Update Optimization (#7853) [Backport-2.x] Cluster State Update Optimization (#7853) Jul 11, 2023
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.remotestore.SegmentReplicationRemoteStoreIT.testCancellation
      1 org.opensearch.indices.replication.SegmentReplicationIT.testScrollCreatedOnReplica
      1 org.opensearch.cluster.allocation.AwarenessAllocationIT.testThreeZoneOneReplicaWithForceZoneValueAndLoadAwareness

@codecov
Copy link

codecov bot commented Jul 11, 2023

Codecov Report

Merging #8644 (61c23f6) into 2.x (b8cebf4) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x    #8644      +/-   ##
============================================
+ Coverage     70.55%   70.58%   +0.02%     
- Complexity    57076    57097      +21     
============================================
  Files          4745     4745              
  Lines        270757   270777      +20     
  Branches      39961    39967       +6     
============================================
+ Hits         191033   191117      +84     
+ Misses        63270    63212      -58     
+ Partials      16454    16448       -6     
Impacted Files Coverage Δ
...java/org/opensearch/cluster/metadata/Metadata.java 87.03% <100.00%> (-0.18%) ⬇️

... and 455 files with indirect coverage changes

@shwetathareja shwetathareja merged commit 5a0b6b0 into opensearch-project:2.x Jul 12, 2023
@sandeshkr419 sandeshkr419 deleted the backport/backport-7853-to-2.x branch July 12, 2023 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants