Create a hook to capture copy batch errors and retries by grodowski · Pull Request #1507 · github/gh-ost

@grodowski marked this pull request as ready for review

March 11, 2025 15:40

Copilot AI review requested due to automatic review settings

March 11, 2025 15:40

grodowski

grodowski

Co-authored-by: Bastian Bartmann <bastian.bartmann@shopify.com>
Co-authored-by: Bastian Bartmann <bastian.bartmann@shopify.com>
CalculateNextIterationRangeEndValues needs to be recomputed on every retry in case of configuration (e.g. chunk-size)
changes were made by onBatchCopyRetry hooks.

@grodowski

@grodowski

@grodowski

…e insert completes

- extract MigrationContext.SetNextIterationRangeValues outside of applyCopyRowsFunc, so that it doesn't run on retries
- add an integration test for Migrator with retry hooks

Co-authored-by: Bastian Bartmann <bastian.bartmann@shopify.com>

@grodowski

@grodowski

@grodowski

@grodowski

@grodowski

@grodowski

@grodowski

@grodowski

meiji163

@grodowski