Thread: Re: [HACKERS] [COMMITTERS] pgsql: Add a Gather executor node.

On Fri, Jul 21, 2017 at 7:06 AM, Andres Freund <andres@anarazel.de> wrote:
> On 2015-09-30 23:29:30 +0000, Robert Haas wrote:
>> Add a Gather executor node.
>> ...
>> src/backend/executor/execProcnode.c           |   46 ++++
>
> I just noticed that this added a new execProcnode dispatch routine, but
> didn't add that to the file's header
>
>  *       INTERFACE ROUTINES
>  *              ExecInitNode    -               initialize a plan node and its subplans
>  *              ExecProcNode    -               get a tuple by executing the plan node
>  *              ExecEndNode             -               shut down a plan node and its subplans
>  *

I think those top-of-file lists are just annoying, and would prefer to
rip them out entirely rather than spend time maintaining them.  If you
want to see the list of interface routines, look in the header file.

heapam.c's header, for example, is missing try_relation_open,
relation_openrv_extended, heap_openrv_extended,
heap_beginscan_catalog, heap_beginscan_strat, heap_beginscan_bm,
heap_beginscan_sampling, heap_setscanlimits, heapgetpage,
heap_rescan_set_params, heap_parallelscan_estimate,
heap_parallelscan_initialize, heap_beginscan_parallel,
heap_hot_search_buffer, heap_hot_search, setLastTid,
GetBulkInsertState, FreeBulkInsertState, ReleaseBulkInsertState,
heap_finish_speculative, heap_abort_speculative, heap_lock_tuple,
heap_inplace_update, heap_tuple_needs_freeze,
heap_tuple_needs_eventual_freeze, simple_heap_insert,
simple_heap_update, simple_heap_delete, and heap_update_snapshot,
which means that if this comment was ever correct, it was more than a
decade ago.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Robert Haas wrote:

> I think those top-of-file lists are just annoying, and would prefer to
> rip them out entirely rather than spend time maintaining them.

+1

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

On July 21, 2017 5:34:32 PM GMT+02:00, Alvaro Herrera <alvherre@2ndquadrant.com> wrote:
>Robert Haas wrote:
>
>> I think those top-of-file lists are just annoying, and would prefer
>to
>> rip them out entirely rather than spend time maintaining them.
>
>+1

I'm quite sympathetic to that view. But I think it's either them, or ripping out, not leaving bit rot.

Andres

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> Robert Haas wrote:
>> I think those top-of-file lists are just annoying, and would prefer to
>> rip them out entirely rather than spend time maintaining them.

> +1

To the extent that they're just lists of function names, +1.  Some of
them have some documentation in them, which you'd need to make sure
is duplicative or else copy it to an appropriate place.
        regards, tom lane