On 2017-05-03 08:17, Petr Jelinek wrote:
On 02/05/17 20:43, Robert Haas wrote:
On Thu, Apr 20, 2017 at 2:58 PM, Peter Eisentraut

code path that calls CommitTransactionCommand() should have one, no?

Is there anything left to be committed here?

Afaics the fix was not committed. Peter wanted more comprehensive fix
which didn't happen. I think something like attached should do the job.

I'm running my pgbench-over-logical-replication test in chunk of 15 minutes, wth different pgbench -c (num clients) and -s (scale) values.

With this patch (and nothing else) on top of master (8f8b9be51fd7 to be precise):


logical replication is still often failing (as expected, I suppose; it seems because of "inital snapshot too large") but indeed I do not see the 'TRAP: FailedAssertion in pgstat.c' anymore.

(If there is any other configuration of patches worth testing please let me know)


Erik Rijkers

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to