From: | Jeff Janes <jeff(dot)janes(at)gmail(dot)com> |
---|---|
To: | Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Problems with plan estimates in postgres_fdw |
Date: | 2019-02-20 21:19:03 |
Message-ID: | CAMkU=1z1Ez7fb_P_0Bc1040npE5fCOnu0M1DFyOzCp=e=rBJCw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Jan 30, 2019 at 6:12 AM Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp>
wrote:
> (2018/12/28 15:50), Etsuro Fujita wrote:
> > Attached is a new version of the
> > patch.
>
> Here is an updated version of the patch set. Changes are:
>
> * In the previous version, LIMIT without OFFSET was not performed
> remotely as the costs was calculated the same as the costs of performing
> it locally. (Actually, such LIMIT was performed remotely in a case in
> the postgres_fdw regression test, but that was due to a bug :(.) I
> think we should prefer performing such LIMIT remotely as that avoids
> extra row fetches from the remote that performing it locally might
> cause, so I tweaked the costs estimated in estimate_path_cost_size(), to
> ensure that we'll prefer performing such LIMIT remotely.
>
With your tweaks, I'm still not seeing the ORDER-less LIMIT get pushed down
when using use_remote_estimate in a simple test case, either with this set
of patches, nor in the V4 set. However, without use_remote_estimate, the
LIMIT is now getting pushed with these patches when it does not in HEAD.
See attached test case, to be run in new database named 'foo'.
Cheers,
Jeff
Attachment | Content-Type | Size |
---|---|---|
fdw_limit.sql | text/plain | 367 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Euler Taveira | 2019-02-20 21:19:34 | Re: BRIN summarize autovac_report_workitem passes datname as relname |
Previous Message | Stephen Frost | 2019-02-20 21:12:02 | Re: Compressed TOAST Slicing |