| From: | Michael Paquier <michael(at)paquier(dot)xyz> | 
|---|---|
| To: | Robert Haas <rhaas(at)postgresql(dot)org> | 
| Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org | 
| Subject: | Re: pgsql: pg_surgery: Try to stabilize regression tests. | 
| Date: | 2020-09-20 13:39:05 | 
| Message-ID: | 20200920133905.GA18931@paquier.xyz | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-committers | 
On Fri, Sep 18, 2020 at 05:27:06PM +0000, Robert Haas wrote:
> pg_surgery: Try to stabilize regression tests.
> 
> According to buildfarm member sungazer, the behavior of VACUUM can be
> unstable in these tests even if we prevent autovacuum from running on
> the tables in question, apparently because even a manual vacuum can
> behave differently depending on whether anything else is running that
> holds back the global xmin. So use a temporary table instead, which
> as of commit a7212be8b9e0885ee769e8c55f99ef742cda487b enables
> vacuuming using a more aggressive cutoff.
> 
> This approach can't be used for the regression test that involves a
> materialized view, but that test doesn't run vacuum, so it shouldn't
> be prone to this particular failure mode.
> 
> Analysis by Tom Lane. Patch by Ashutosh Sharma and me.
rorqual has just reported an issue here:
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=rorqual&dt=2020-09-20%2011%3A20%3A11
 vacuum freeze htab2;
+ERROR:  cannot freeze committed xmax 23257
+CONTEXT:  while scanning block 0 and offset 2 of relation "pg_temp_3.htab2"
 -- unused TIDs should be skipped
 select heap_force_kill('htab2'::regclass, ARRAY['(0, 2)']::tid[]);
-NOTICE:  skipping tid (0, 2) for relation "htab2" because it is marked unused 
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2020-09-20 13:47:17 | pgsql: Fix whitespace | 
| Previous Message | Bruce Momjian | 2020-09-20 03:19:53 | Re: pgsql: Doc: fill in "major enhancements" list in v13 release notes. |