From: | Petr Jelinek <petr(at)2ndquadrant(dot)com> |
---|---|
To: | Kouhei Kaigai <kaigai(at)ak(dot)jp(dot)nec(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Reworks of CustomScan serialization/deserialization |
Date: | 2016-03-23 17:36:48 |
Message-ID: | 56F2D430.300@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 23/03/16 09:14, Kouhei Kaigai wrote:
>>>> On 15/03/16 05:03, Kouhei Kaigai wrote:
>>>>> Petr,
>>>>>
>>>>> The attached patch is the revised one that follows the new extensible-
>>>>> node routine.
>>>>>
>>>>> It is almost same the previous version except for:
>>>>> - custom-apis.[ch] was renamed to custom-node.[ch]
>>>>> - check for the length of custom-scan-method name followed
>>>>> the manner of RegisterExtensibleNodeMethods()
>>>>>
>>>>
>>>> Hi,
>>>>
>>>> looks good, only nitpick I have is that it probably should be
>>>> custom_node.h with underscore given that we use underscore everywhere
>>>> (except for libpq and for some reason atomic ops).
>>>>
>>> Sorry for my response late.
>>>
>>> The attached patch just renamed custom-node.[ch] by custom_node.[ch].
>>> Other portions are not changed from the previous revison.
>>>
>>
>> Forgot to attach?
>>
> Yes.... Thanks,
Ok, I am happy with it, marked it as ready for committer (it was marked
as committed although it wasn't committed).
--
Petr Jelinek http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2016-03-23 17:37:28 | Re: PostgreSQL 9.6 behavior change with set returning (funct).* |
Previous Message | Magnus Hagander | 2016-03-23 17:32:11 | Re: Rationalizing code-sharing among src/bin/ directories |