Skip to content

Adapt applier to support DML events for move tables#1691

Draft
danieljoos wants to merge 1 commit into
move-tables/applier-apply-move-table-queriesfrom
move-tables/applier-dml-events
Draft

Adapt applier to support DML events for move tables#1691
danieljoos wants to merge 1 commit into
move-tables/applier-apply-move-table-queriesfrom
move-tables/applier-dml-events

Conversation

@danieljoos
Copy link
Copy Markdown
Contributor

@danieljoos danieljoos commented May 27, 2026

A Pull Request should be associated with an Issue.

We wish to have discussions in Issues. A single issue may be targeted by multiple PRs.
If you're offering a new feature or fixing anything, we'd like to know beforehand in Issues,
and potentially we'll be able to point development in a particular direction.

Related issue (public): #1681
Related issue (internal): https://github.com/github/database-infrastructure/issues/8175

Further notes in https://github.com/github/gh-ost/blob/master/.github/CONTRIBUTING.md
Thank you! We are open to PRs, but please understand if for technical reasons we are unable to accept each and any PR

Description

This PR is part of the move-tables feature development PR stack.
It adapts the Applier to write DML events to the target database when in move-tables mode.

In case this PR introduced Go code changes:

  • contributed code is using same conventions as original code
  • script/cibuild returns with no formatting errors, build errors or unit test errors.

@danieljoos danieljoos changed the title Adapt applier to support DML events for move tables [move-tables] Adapt applier to support DML events for move tables May 27, 2026
@danieljoos danieljoos changed the title [move-tables] Adapt applier to support DML events for move tables Adapt applier to support DML events for move tables May 27, 2026
@danieljoos danieljoos added the feature-move-tables PRs that are associated with the new move-tables feature label May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature-move-tables PRs that are associated with the new move-tables feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant