"Matthew T. O'Connor" <matthew@zeut.net> writes: > Tom Lane wrote: >> Surely this is completely broken? AFAICT you are testing the result >> from a VACUUM or ANALYZE command, which is not going to return any >> tuples.
> Upon further inspection, I think you are right. I would think that > instead of checking the query result with PQntuples, it should probably > be checked with |PQresultStatus. ISTM this is the wrong place to test at all. I put a PQntuples check into update_table_thresholds() instead, which seems a far more direct defense against trouble. (Consider eg the case where someone drops the table just after your VACUUM completes successfully. Also there are drop/rename scenarios to think about: success of the VACUUM proves that there is a table named FOO, not that there is still a table with the OID you have on record.) regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 3: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faq