Add toxiproxy localtest by meiji163 · Pull Request #1591 · github/gh-ost
marked this pull request as ready for review
Copilot AI review requested due to automatic review settings
October 10, 2025 03:06
meiji163
changed the title
Add toxiproxy localtests
Add toxiproxy localtest
meiji163 added a commit that referenced this pull request
Oct 10, 2025
…ly (#1594) * Panic if InitiateHeartbeat exhausts retries to avoid looping infinitely. Based on experience, if the writer database fails inbeetween the copy & cutover stages (e.g. during cutover pause), the heartbeat writes will fail and stop, then leading to throttled state and an infinite loop of throttler.shouldThrottle(). Since this state is irrecoverable, make the heartbeat writer panic if retries are exhausted, so that the migration can fail and be restarted later. * Add sysbench localtest (#1590) * add sysbench localtest * fix table name * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * ensure cleanup --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * add toxiproxy option for localtests (#1591) --------- Co-authored-by: Jan Grodowski <jan.grodowski@shopify.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters