| From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
|---|---|
| To: | Michael Paquier <michael(at)paquier(dot)xyz> |
| Cc: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, John Naylor <john(dot)naylor(at)enterprisedb(dot)com>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Ian Lawrence Barwick <barwick(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Anastasia Lubennikova <lubennikovaav(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Peter Geoghegan <pg(at)bowt(dot)ie> |
| Subject: | Re: Testing autovacuum wraparound (including failsafe) |
| Date: | 2023-08-25 09:26:26 |
| Message-ID: | FBE72C6F-E64C-46D0-ABCC-14E3303EFD7A@yesql.se |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On 22 Aug 2023, at 07:49, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Wed, Jul 12, 2023 at 01:47:51PM +0200, Daniel Gustafsson wrote:
>> +# bump the query timeout to avoid false negatives on slow test syetems.
>> +$ENV{PG_TEST_TIMEOUT_DEFAULT} = 600;
>> Does this actually work? Utils.pm read the environment variable at compile
>> time in the BEGIN block so this setting won't be seen? A quick testprogram
>> seems to confirm this but I might be missing something.
>
> I wish that this test were cheaper, without a need to depend on
> PG_TEST_EXTRA.. Actually, note that you are forgetting to update the
> documentation of PG_TEST_EXTRA with this new value of xid_wraparound.
Agreed, it would be nice, but I don't see any way to achieve that. I still
think the test is worthwhile to add, once the upthread mentioned issues are
resolved.
--
Daniel Gustafsson
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Anthony Roberts | 2023-08-25 09:40:39 | Re: [PATCH] Add native windows on arm64 support |
| Previous Message | Richard Guo | 2023-08-25 08:48:35 | Re: Support run-time partition pruning for hash join |