pgsql: YA attempt to stabilize the results of the postgres_fdw regressi

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: YA attempt to stabilize the results of the postgres_fdw regressi
Date: 2018-04-12 19:12:23
Message-ID: E1f6hdv-0004Mj-OU@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

YA attempt to stabilize the results of the postgres_fdw regression test.

We've made multiple attempts to stabilize the plans shown by commit
1bc0100d2, with little success so far. The reason for the remaining
instability seems to be that if a transaction (such as auto-analyze)
is running concurrently with the test, then get_actual_variable_range may
return a maximum value for "T 1"."C 1" that's far away from the actual max,
as a result of our having transiently inserted such a value earlier in
the test. Because we use a non-MVCC snapshot to fetch the value (for
performance reasons), the presence of other transactions can cause that
function to return entries that are actually dead.

To fix, use a less extreme value in the earlier transient insertion, so
that whether it is visible or not won't affect the selectivity estimate.
The use of 9999 there seems to have been picked with the aid of a
dartboard anyway, rather than having a specific reason.

Discussion: https://postgr.es/m/16962.1523551784@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/2fe977712c7375ccb1b6ddf7dfb234d0db903f16

Modified Files
--------------
contrib/postgres_fdw/expected/postgres_fdw.out | 18 +++++++++---------
contrib/postgres_fdw/sql/postgres_fdw.sql | 12 ++++++------
2 files changed, 15 insertions(+), 15 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2018-04-12 19:52:54 pgsql: Add comment about default partition in check_new_partition_bound
Previous Message Alvaro Herrera 2018-04-12 18:09:53 pgsql: Use the right memory context for partkey's FmgrInfo