From: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: pgsql: Support partition pruning at execution time |
Date: | 2018-04-10 15:50:43 |
Message-ID: | CAKJS1f_Y4LGYKRZtKfOPHp7zXZRMeHg0-J04HXTuntWiUG5S6Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On 11 April 2018 at 03:14, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote:
> 2. Do we want to revert Andrew's test stabilization patch? If I
> understand correctly, the problem is the inverse of what was diagnosed:
> "any running transaction at the time of the test could prevent pages
> from being set as all-visible". That's correct, but the test doesn't
> depend on pages being all-visible -- quite the contrary, it depends on
> the pages NOT being all-visible (which is why the HeapFetches counts are
> all non-zero). Since the pages contain very few tuples, autovacuum
> should never process the tables anyway.
I think it's probably a good idea to revert it once the
instrumentation is working correctly. It appears this found a bug in
that code, so is probably useful to keep just in case something else
breaks it in the future.
I don't think there is too much risk of instability from other
sources. There's no reason an auto-vacuum would trigger and cause a
change in heap fetches. We only delete one row from lprt_a, that's not
going to trigger an auto-vacuum.
--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2018-04-10 15:56:56 | Re: pgsql: Support partition pruning at execution time |
Previous Message | Tom Lane | 2018-04-10 15:42:34 | Re: pgsql: Support partition pruning at execution time |
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2018-04-10 15:56:56 | Re: pgsql: Support partition pruning at execution time |
Previous Message | Tom Lane | 2018-04-10 15:42:34 | Re: pgsql: Support partition pruning at execution time |