]> granicus.if.org Git - postgresql/commitdiff
Suppress scary-looking log messages from async-notify isolation test.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 1 Mar 2016 00:29:19 +0000 (19:29 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 1 Mar 2016 00:29:19 +0000 (19:29 -0500)
I noticed that the async-notify test results in log messages like these:

LOG:  could not send data to client: Broken pipe
FATAL:  connection to client lost

This is because it unceremoniously disconnects a client session that is
about to have some NOTIFY messages delivered to it.  Such log messages
during a regression test might well cause people to go looking for a
problem that doesn't really exist (it did cause me to waste some time that
way).  We can shut it up by adding an UNLISTEN command to session teardown.

Patch HEAD only; this doesn't seem significant enough to back-patch.

src/test/isolation/specs/async-notify.spec

index 7f451b18a1579e808abb36ba6ed127c3571a12f5..8adad42c7c846b704443d5657081df14af30074d 100644 (file)
@@ -5,7 +5,7 @@
 session "listener"
 step "listen"  { LISTEN a; }
 step "begin"   { BEGIN; }
-teardown               { ROLLBACK; }
+teardown               { ROLLBACK; UNLISTEN *; }
 
 session "notifier"
 step "check"   { SELECT pg_notification_queue_usage() > 0 AS nonzero; }