Re: postgres_fdw: perform UPDATE/DELETE .. RETURNING on a join directly
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
Cc: Etsuro Fujita <fujita.etsuro@lab.ntt.co.jp>,
Robert Haas <robertmhaas@gmail.com>,
Pg Hackers <pgsql-hackers@postgresql.org>,
Rushabh Lathia <rushabh.lathia@gmail.com>,
Michael Paquier <michael.paquier@gmail.com>
Date: 2018-03-05T05:51:52Z
Lists: pgsql-hackers
Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> writes: > On Mon, Mar 5, 2018 at 8:51 AM, Etsuro Fujita > <fujita.etsuro@lab.ntt.co.jp> wrote: >> Totally outdated stats used in query planning causes the failures? ANALYZE >> right before the plan-changing queries would fix the failures? > I think the testcase file for postgres_fdw has grown really large over > the time, as we have added more pushdown. Since most of the queries in > that file use the same tables created at the beginning of the file, > changes somewhere in-between (esp. DMLs) affect the following queries. > It's getting hard to add a test query in that file and expect stable > results. The thing that I find curious, now that we've shut off autovacuum altogether on those tables, is that we *still* aren't getting stable results. How can that be? Yeah, if you add another query in the middle you might find that changing later results, but for any fixed contents of the test script, why isn't the buildfarm getting the same answers that the patch author and committer got? I could believe different results on 32-bit and 64-bit hardware, but that does not seem to be quite the pattern we're seeing. > I think, it's time to split the file into at least two one > for queries and one for DMLs. That seems largely irrelevant right at this point. First we have to explain the instability. regards, tom lane
Commits
-
Temporarily instrument postgres_fdw test to look for statistics changes.
- c2c537c56dc3 11.0 landed
-
postgres_fdw: Fourth attempt to stabilize regression tests.
- 1733460f0205 11.0 landed
-
postgres_fdw: Third attempt to stabilize regression tests.
- 4fa396464e5f 11.0 landed
-
postgres_fdw: Attmempt to stabilize regression tests.
- 958e20e42d6c 11.0 landed
-
postgres_fdw: Push down UPDATE/DELETE joins to remote servers.
- 1bc0100d270e 11.0 landed