On Thu, Sep 8, 2016 at 10:41 PM, Ashutosh Bapat <
>> While checking for shippability, we build the target list which is passed
>> the foreign server as fdw_scan_tlist. The target list contains
>> a. All the GROUP BY expressions
>> b. Shippable entries from the target list of upper relation
>> c. Var and Aggref nodes from non-shippable entries from the target list of
>> upper relation
> The code in the patch doesn't seem to add Var nodes explicitly. It assumes
> the Var nodes will be part of GROUP BY clause. The code is correct, I
Yes. Code is correct. Var nodes are already part of GROUP BY else we hit
error well before this point.
Thanks Ashutosh for the detailed review comments.
I am working on it and will post updated patch once I fix all your concerns.
Jeevan B Chalke
Principal Software Engineer, Product Development
The Enterprise PostgreSQL Company