Hi Tim,
I have to correct my previous answer, my change does not alter the behavior of your patch significantly.
> The difference:
> In your version of the patch vacuumlo will stop after N lo_unlink(OID) attempts.
> The previous behavior of the patch is that vacuumlo will stop after N successful lo_unlink(OID)s.
>
> If you have good reason for your behavior please add another flag so that it is optional.
> There should be a clear distinction between "counting vs not", and "aborting vs continuing" when a lo_unlink(OID) is
unsuccessful.
if (param->dry_run == 0) { if (lo_unlink(conn, lo) < 0)
{ fprintf(stderr, "\nFailed to remove lo %u: ", lo);
fprintf(stderr, "%s", PQerrorMessage(conn)); } else
deleted++; } else deleted++;
if(param->transaction_limit!=0&& deleted>=param->transaction_limit) break;
The variable "deleted" is only incremented if a lo_unlink was successful, so my patch only introduces a negligible
overheadbut no actual change in behavior.
I'm very grateful for your patch and I think it should be accepted as soon as possible, one or two "if" does not matter
tome.
Aron
