Skip to content

Fix BBM testing failing with zero rows to migrate

Simon Tomlinson requested to merge bbm-testing-empty-ending-batch into master

What does this MR do and why?

Fixes a bug in batched background migration sampling where, if a migration's filter condition didn't match any rows from a sampled batch start to the end of the table, we would attempt to test a batch with nil bounds.

Skip these empty batches instead, mirroring the behavior of real background migration testing where we would finish the migration rather than running another batch.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Simon Tomlinson

Merge request reports