| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
| Cc: | pgsql-committers(at)postgresql(dot)org |
| Subject: | Re: pgsql: Add support for temporary replication slots |
| Date: | 2016-12-12 15:55:44 |
| Message-ID: | 27833.1481558144@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
I wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
>> Add support for temporary replication slots
> Some of the slower buildfarm members are failing test-decoding-check since
> this went in. At least on prairiedog, it looks like a race condition in
> the test:
Having now looked a bit more closely, that's almost certainly what it is:
the test is assuming that the old backend exits instantaneously, which
it doesn't. You might want to borrow the wait-for-previous-session-to-die
loop I put into src/test/modules/test_extensions/sql/test_extensions.sql
recently. (Make sure you get the fixed version ;-))
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2016-12-12 16:23:38 | pgsql: Fix creative, but unportable, spelling of "ptr != NULL". |
| Previous Message | Tom Lane | 2016-12-12 15:43:00 | Re: pgsql: Add support for temporary replication slots |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2016-12-12 16:19:21 | Re: exposing wait events for non-backends (was: Tracking wait event for latches) |
| Previous Message | Tom Lane | 2016-12-12 15:43:00 | Re: pgsql: Add support for temporary replication slots |