Skip to content

Conversation

@JeremyDahlgren
Copy link
Contributor

Relocates all Settings from RemoteClusterService to a RemoteClusterSettings class.
Note that a RemoteClusterSettingsTests class already exists that tests some of these settings.
This is one part in a series of refactors to use a LinkedProjectConfig data class in RemoteClusterService and its related classes. Follow up PRs will relocate additional settings from related RemoteClusterService classes into RemoteClusterSettings.

Relates: ES-12656, ES-12569

Relocates all Settings from RemoteClusterService to a
RemoteClusterSettings class.  Note that a
RemoteClusterSettingsTests class already exists that tests some of
these settings. This is one part in a series of refactors to use a
LinkedProjectConfig data class in RemoteClusterService and its related
classes.

Relates: ES-12656, ES-12569
@JeremyDahlgren JeremyDahlgren added :Distributed Coordination/Network Http and internode communication implementations >refactoring Team:Distributed Coordination Meta label for Distributed Coordination team v9.2.0 labels Aug 18, 2025
@JeremyDahlgren JeremyDahlgren requested a review from ywangd August 18, 2025 21:12
@JeremyDahlgren JeremyDahlgren marked this pull request as ready for review August 18, 2025 21:12
@JeremyDahlgren JeremyDahlgren requested a review from a team as a code owner August 18, 2025 21:12
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@JeremyDahlgren JeremyDahlgren merged commit 3854e38 into elastic:main Aug 19, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Network Http and internode communication implementations >refactoring Team:Distributed Coordination Meta label for Distributed Coordination team v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants