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

[Remote State] Create a RemoteObjectManager interface which is used by all managers #14176

Closed
shiv0408 opened this issue Jun 11, 2024 · 0 comments · Fixed by #14671
Closed

[Remote State] Create a RemoteObjectManager interface which is used by all managers #14176

shiv0408 opened this issue Jun 11, 2024 · 0 comments · Fixed by #14671
Assignees
Labels
Cluster Manager ClusterManager:RemoteState enhancement Enhancement or improvement to existing feature or request

Comments

@shiv0408
Copy link
Member

Is your feature request related to a problem? Please describe

We have added RemoteClusterStateAttributesManager in a9d2050 and other managers are also getting added in #13924. These managers contain some common methods like getAsyncWriteAction, getAsyncReadAction which should be moved to a interface.

Describe the solution you'd like

Create an interface for all the entity managers with defined contract.

Related component

Cluster Manager

Describe alternatives you've considered

No response

Additional context

No response

@shiv0408 shiv0408 added enhancement Enhancement or improvement to existing feature or request untriaged labels Jun 11, 2024
@shwetathareja shwetathareja changed the title [Feature Request] Create a RemoteObjectManager interface with is used by all managers [Remote State] Create a RemoteObjectManager interface with is used by all managers Jun 11, 2024
@shiv0408 shiv0408 changed the title [Remote State] Create a RemoteObjectManager interface with is used by all managers [Remote State] Create a RemoteObjectManager interface which is used by all managers Jun 18, 2024
@shiv0408 shiv0408 self-assigned this Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cluster Manager ClusterManager:RemoteState enhancement Enhancement or improvement to existing feature or request
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

3 participants