Comment 2 for bug 979092

Revision history for this message
Daniel Nichter (daniel-nichter) wrote :

The patch is only safe with a bulk operation like --bulk-delete. The problem is this: the tool has two modes of operation per-row and bulk archiving. --sleep is meant for per-row, to make the tool do:

* SELECT row
* Archive that row
* --sleep

So, this is mode is slow, careful nibbling/archiving without overloading the server. The other mode does:

* SELECT row
* Repeat until last row
* Bulk archive first-last row

Then the tool should --sleep after the bulk archive, which the patch makes happens. But currently the tool is doing --sleep between fetching each row, but it doesn't need to because it's not operating on those rows, it's just trying to advance to the last row so it can do the bulk archive. Then it should sleep (between bulk operations).