Skip to content

Conversation

@ImeevMA
Copy link
Contributor

@ImeevMA ImeevMA commented Dec 22, 2025

This patch adds a new DDL RPC that allows CDC to be restarted on a space.

Needed for tarantool/aeon#589

This patch adds a new DDL RPC that allows CDC to be restarted
on a space.

Needed for tarantool/aeon#589
@ImeevMA ImeevMA force-pushed the imeevma/gh-589-add-cdc-restart branch from b1a9112 to 6771787 Compare December 22, 2025 15:23
@ImeevMA ImeevMA requested review from locker and unera December 22, 2025 15:56
rpc DropSpace(DropSpaceRequest) returns (DropSpaceResponse) {}

// Restart CDC for a space.
rpc RestartCDC(RestartCDCRequest) returns (RestartCDCResponse) {}
Copy link
Member

@locker locker Dec 22, 2025

Choose a reason for hiding this comment

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

We don't need it, actually. We only need the SQL API because this is a command that's supposed to be used by the admin. That's why I mentioned only SQL in the ticket.

// Drops a space by name.
rpc DropSpace(DropSpaceRequest) returns (DropSpaceResponse) {}

// Restart CDC for a space.
Copy link
Member

Choose a reason for hiding this comment

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

Unclear what CDC restart does. It'd probably be nice to say a few words about it, as well as about what messages are pushed and when if CDC is enabled for a space.

@locker locker assigned ImeevMA and unassigned locker Dec 22, 2025
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.

4 participants