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

hooks: reporting GH_OST_ETA_SECONDS. ETA as part of migration context #936

Merged
merged 30 commits into from
May 31, 2021

Conversation

shlomi-noach
Copy link
Contributor

resubmission of openark#17

Hooks can now utilize GH_OST_ETA_SECONDS environment variable, populated with an integer value of ETA seconds.

A negative value stands for unknown ETA, as is the case at the very beginning of the migration.
The value is 0 when cut-over is imminent.


maintainers: this joins a list of unmerged upstream PRs while I'm progressing with downstream development. It will be difficult to identify the specific changes in this PR given the unmerged commits. But do see openark#17

Support a complete ALTER TABLE statement in --alter
Initial commit: towards setting up a test suite

Signed-off-by: Shlomi Noach <[email protected]>
…original table, applying AUTO_INCREMENT value onto ghost table if applicable and user has not specified AUTO_INCREMENT in alter statement
Signed-off-by: Shlomi Noach <[email protected]>
Signed-off-by: Shlomi Noach <[email protected]>
Copying AUTO_INCREMENT value to ghost table
Generated column as part of UNIQUE (or PRIMARY) KEY
Cut-over should wait for heartbeat lag to be low enough to succeed
All MySQL DBs limited to max 3 concurrent/idle connections
Signed-off-by: Shlomi Noach <[email protected]>
Signed-off-by: Shlomi Noach <[email protected]>
Signed-off-by: Shlomi Noach <[email protected]>
@timvaillancourt timvaillancourt added this to the v1.1.2 milestone May 6, 2021
@shlomi-noach
Copy link
Contributor Author

branch updated after recent merge of downstream contribution.

@timvaillancourt
Copy link
Collaborator

This PR passed many internal tests, merging it 👍

Thanks @shlomi-noach!

@timvaillancourt timvaillancourt merged commit f19f101 into github:master May 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants