ctdb-daemon: Do not force full vacuum on first vacuuming run
authorMartin Schwenke <martin@meltin.net>
Mon, 22 Oct 2018 10:40:22 +0000 (21:40 +1100)
committerAmitay Isaacs <amitay@samba.org>
Tue, 18 Dec 2018 06:12:09 +0000 (07:12 +0100)
commitf1b594dce1c081924b16c52c8c50c984a16df098
tree014e3cf766818c20db1f402b11b1498cf00d66c0
parent9bdd6814e4c64f3be04daa7e8739e141d2d664b7
ctdb-daemon: Do not force full vacuum on first vacuuming run

When the number of fast path vacuuming runs is 0 then a full vacuuming
run is done.  This means the first one is a full run, which is almost
certainly not what is intended.

Combine the 2 conditionals to only flag a full vacuuming run when the
count exceeds the configured limit.  This means that the
full_vacuum_run flag is set in both parent and child, but this is
harmless... and is better than getting it wrong.

Also tweak the comparison to be less-than-or-equal, since the zeroth
run needs to be counted.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
ctdb/server/ctdb_vacuum.c