Minor robustness improvements for isolationtester.

Tom Lane <tgl@sss.pgh.pa.us>

Commit: 845d335a90b684dd51e80a6470ebb923a59a1f91
Author: Tom Lane <tgl@sss.pgh.pa.us>
Date: 2013-04-03T01:15:37Z
Releases: 9.3.0
Minor robustness improvements for isolationtester.

Notice and complain about PQcancel() failures.  Also, don't dump core if
an error PGresult doesn't contain severity and message subfields, as it
might not if it was generated by libpq itself.  (We have a longstanding
TODO item to improve that, but in the meantime isolationtester had better
cope.)

I tripped across the latter item while investigating a trouble report on
buildfarm member spoonbill.  As for the former, there's no evidence that
PQcancel failure is actually involved in spoonbill's problem, but it still
seems like a bad idea to ignore an error return code.

Files

PathChange+/−
src/test/isolation/isolationtester.c modified +22 −9