| From: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> |
|---|---|
| To: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | Jeevan Chalke <jeevan(dot)chalke(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Aggregate Push Down - Performing aggregation on foreign server |
| Date: | 2016-08-31 08:03:04 |
| Message-ID: | 0ff3b5fc-bc75-b55d-9a59-9e9b728ee3b9@lab.ntt.co.jp |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 2016/08/31 16:42, Pavel Stehule wrote:
> 2016-08-31 9:00 GMT+02:00 Robert Haas <robertmhaas(at)gmail(dot)com>:
>
>> On Wed, Aug 31, 2016 at 11:56 AM, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
>> wrote:
>>> It is pity - lot of performance issues are related to this missing
>> feature.
>>
>> I don't think you are being very clear about what feature you are
>> talking about. The feature that Jeevan has implemented is pushing
>> aggregates to the remote side when postgres_fdw is in use. The
>> feature you are talking about is evidently something else, but you
>> haven't really said what it is. Or not in a way that I can
>> understand.
>>
>
> yes, It is not clear if FDW aggregate push down can be implemented together
> with internal aggregate push down. Aggregate push down ~ try to aggregate
> first when it is possible
What do you mean by "internal aggregate push down"? Partition-wise
aggregation? Aggregate/group by before join (something like [1])? IIUC,
what Jeevan proposes in this thread is to implement the aggregate push
down API that is specific to FDWs in postgres_fdw. Any other push down
work would need to use different APIs and would need to separately
proposed/discussed.
Thanks,
Amit
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jeevan Chalke | 2016-08-31 08:09:32 | Re: Small patch for snapmgr.c |
| Previous Message | Michael Paquier | 2016-08-31 07:57:20 | Replace use malloc() & friend by memory contexts for plperl and pltcl |