Skip to content

IGNITE-22896 REST API for initiating MG repair#4547

Merged
rpuch merged 6 commits intoapache:mainfrom
gridgain:ignite-22896
Oct 15, 2024
Merged

IGNITE-22896 REST API for initiating MG repair#4547
rpuch merged 6 commits intoapache:mainfrom
gridgain:ignite-22896

Conversation

@Phillippko
Copy link
Contributor

@Phillippko Phillippko marked this pull request as ready for review October 14, 2024 09:04
@rpuch rpuch merged commit cc53fb6 into apache:main Oct 15, 2024
@rpuch rpuch deleted the ignite-22896 branch October 15, 2024 14:45
@Schema(description = "Reset cluster.")
public class ResetClusterRequest {
@Schema(description = "Names of the proposed CMG nodes.")
@Schema(description = "Names of the proposed CMG nodes. Optional if Metastorage replication factor is specified, then "
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure what replication factor has to do with it... Re-phrase for clarity... Discuss?

private final List<String> cmgNodeNames;
private final @Nullable List<String> cmgNodeNames;

@Schema(description = "Number of nodes in the Raft voting member set for Metastorage.")
Copy link
Contributor

Choose a reason for hiding this comment

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

Number of nodes in the voting member set of the Metastorage RAFT group.

* Initiates cluster reset. CMG will be reset and Metastorage will be repaired.
*
* @param proposedCmgNodeNames Names of the nodes that will be the new CMG nodes.
* @param proposedCmgNodeNames Names of the nodes that will be the new CMG nodes. If not specified, current CMG nodes will be used.
Copy link
Contributor

Choose a reason for hiding this comment

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

Names of the new CMG nodes. If not specified, the current CMG nodes are used.

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.

3 participants