]> granicus.if.org Git - postgresql/commit
Minor robustness improvements for isolationtester.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 3 Apr 2013 01:15:50 +0000 (21:15 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 3 Apr 2013 01:15:50 +0000 (21:15 -0400)
commit677fde06afd23f355e3d599eedc3ba7a66ef4e85
tree0cc39e6631efcb3a979c11006ef6a070641a3166
parent114fca526e4f843d17f9a052f81f580d9d006ef1
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.
src/test/isolation/isolationtester.c