Skip to content

[Backport 2.x] Fix circular dependency in Settings initialization#10317

Merged
reta merged 2 commits into2.xfrom
backport/backport-10194-to-2.x
Oct 4, 2023
Merged

[Backport 2.x] Fix circular dependency in Settings initialization#10317
reta merged 2 commits into2.xfrom
backport/backport-10194-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport d3bf230 from #10194.

There was a weird circular class-loading dependency between Settings and
Settings.Builder that could result in ClassLoader deadlock if multiple
threads trigger class-loading of Settings at the same time.

Signed-off-by: Michael Froh <[email protected]>
(cherry picked from commit d3bf230)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Oct 2, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 2, 2023

Compatibility status:

Checks if related components are compatible with change 1f624fe

Incompatible components

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/neural-search.git]

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Oct 3, 2023

Codecov Report

Merging #10317 (1f624fe) into 2.x (2f430ed) will increase coverage by 0.03%.
Report is 6 commits behind head on 2.x.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x   #10317      +/-   ##
============================================
+ Coverage     70.87%   70.91%   +0.03%     
- Complexity    58354    58406      +52     
============================================
  Files          4816     4816              
  Lines        275763   275763              
  Branches      40534    40534              
============================================
+ Hits         195440   195544     +104     
+ Misses        63664    63536     -128     
- Partials      16659    16683      +24     
Files Coverage Δ
.../java/org/opensearch/common/settings/Settings.java 80.62% <100.00%> (-0.20%) ⬇️

... and 458 files with indirect coverage changes

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 4, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.search.pit.DeletePitMultiNodeIT.testDeleteWhileSearch
      1 org.opensearch.index.shard.RemoteIndexShardTests.testRepicaCleansUpOldCommitsWhenReceivingNew

@reta reta merged commit 6170dd9 into 2.x Oct 4, 2023
@reta reta added the v2.11.0 Issues and PRs related to version 2.11.0 label Oct 4, 2023
@github-actions github-actions bot deleted the backport/backport-10194-to-2.x branch October 4, 2023 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v2.11.0 Issues and PRs related to version 2.11.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants