Re: postgres_fdw: using TABLESAMPLE to collect remote sample
Tomas Vondra <tomas.vondra@enterprisedb.com>
From: Tomas Vondra <tomas.vondra@enterprisedb.com>
To: Robert Haas <robertmhaas@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2022-02-11T19:52:46Z
Lists: pgsql-hackers
On 2/11/22 19:13, Robert Haas wrote: > On Fri, Feb 11, 2022 at 1:06 PM Tom Lane <tgl@sss.pgh.pa.us> wrote: >> While I'm not opposed to moving those goalposts at some point, >> I think pushing them all the way up to 9.5 for this one easily-fixed >> problem is not very reasonable. >> >> Given other recent discussion, an argument could be made for moving >> the cutoff to 9.2, on the grounds that it's too hard to test against >> anything older. But that still leaves us needing a version check >> if we want to use TABLESAMPLE. > > OK, thanks. > Yeah, I think checking server_version is fairly simple. Which is mostly why I haven't done anything about that in the submitted patch, because the other issues (what fraction to sample) seemed more important. regards -- Tomas Vondra EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
Commits
-
Check relkind before using TABLESAMPLE in postgres_fdw
- 57d11ef028d1 16.0 landed
-
Fix stale comment about sample_frac adjustment
- 211d80c06562 16.0 landed