Re: Increasing timeout of poll_query_until for TAP tests

2018-01-01 Thread Noah Misch
On Mon, Jan 01, 2018 at 07:55:37PM +0900, Michael Paquier wrote: > On Sun, Dec 31, 2017 at 09:52:27PM -0800, Noah Misch wrote: > > Since now() is transaction_timestamp(), $recovery_time precedes or equals > > $lsn3, and this didn't close the race. Using clock_timestamp() here would > > work, as do

Re: Increasing timeout of poll_query_until for TAP tests

2018-01-01 Thread Michael Paquier
On Sun, Dec 31, 2017 at 09:52:27PM -0800, Noah Misch wrote: > Since now() is transaction_timestamp(), $recovery_time precedes or equals > $lsn3, and this didn't close the race. Using clock_timestamp() here would > work, as does using separate transactions like recovery-test-fixes.patch did. > I'll

Re: Increasing timeout of poll_query_until for TAP tests

2017-12-31 Thread Noah Misch
On Tue, Aug 02, 2016 at 06:21:03PM -0400, Alvaro Herrera wrote: > Michael Paquier wrote: > > $node_master->safe_psql('postgres', > > "INSERT INTO tab_int VALUES (generate_series(1001,2000))"); > > my $recovery_txid = > > $node_master->safe_psql('postgres', "SELECT txid_current()"); > > my