From: | Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Corey Huinker <corey(dot)huinker(at)gmail(dot)com>, Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, Matt Kelly <mkellycs(at)gmail(dot)com> |
Subject: | Re: [POC] FETCH limited by bytes. |
Date: | 2016-02-03 16:17:06 |
Message-ID: | CAFjFpRfU9_DnDccdZDwzjK70my18HsnwGadDQHsEq_LFbpA8Eg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
There seems to be double successive assignment to fdw_private in recent
commit. Here's patch to remove the first one.
On Wed, Feb 3, 2016 at 7:40 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Tue, Feb 2, 2016 at 10:42 PM, Corey Huinker <corey(dot)huinker(at)gmail(dot)com>
> wrote:
> >> I don't see how. There really is no declaration in there for a
> >> variable called server.
> >
> > Absolutely correct. My only guess is that it was from failing to make
> clean
> > after a checkout/re-checkout. A good reason to have even boring
> regression
> > tests.
> >
> > Regression tests added.
>
> Committed.
>
> --
> Robert Haas
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
--
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company
Attachment | Content-Type | Size |
---|---|---|
pg_double_assignment.patch | text/plain | 1.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2016-02-03 16:36:45 | Re: PostgreSQL Audit Extension |
Previous Message | Robert Haas | 2016-02-03 16:01:55 | Re: CustomScan in a larger structure (RE: CustomScan support on readfuncs.c) |