Here's patch to remove this declaration. The Assert next probably prevents the warning for build with asserts. But both those lines are not needed.
On Wed, Feb 10, 2016 at 12:01 PM, Jeff Janes <jeff.ja...@gmail.com> wrote: > On Tue, Feb 9, 2016 at 11:16 AM, Robert Haas <robertmh...@gmail.com> > wrote: > > On Tue, Feb 9, 2016 at 8:39 AM, Ashutosh Bapat > > <ashutosh.ba...@enterprisedb.com> wrote: > >> Thanks Jeevan for your review and comments. PFA the patch which fixes > those. > > > > Committed with a couple more small adjustments. > > I'm getting a compiler warning which I think is coming from this commit. > > postgres_fdw.c: In function 'fetch_more_data': > postgres_fdw.c:2526:17: warning: unused variable 'fsplan' > [-Wunused-variable] > ForeignScan *fsplan = (ForeignScan *) node->ss.ps.plan; > > Thanks, > > Jeff > -- Best Wishes, Ashutosh Bapat EnterpriseDB Corporation The Postgres Database Company
-- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers