Re: [COMMITTERS] pgsql: Add missing support for new node fields

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andres Freund <andres(at)anarazel(dot)de>, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Add missing support for new node fields
Date: 2017-03-21 16:19:40
Message-ID: CAFj8pRDF-dwq4YhYdAzYd-s6hC4eHgDxcoSGFqSSmkki9BVWwg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

>
> IMHO, what would be a lot more useful than something that generates
> {read,equal,copy,out}funcs.c automatically would be something that
> just checks them for trivial errors of omission. For example, if you
> read a list of structure members from the appropriate header files and
> cross-checked it against the list of structure members mentioned in
> the body of a copy/equal/read/outfunc, you'd catch probably 80% of the
> mistakes people make right there. If you could also check for a
> copy/read/equal/outfunc that ought to have been present but was
> omitted entirely, that'd probably up it to 90%.
>
>
+1

The work on nodes maintenance is not too much, but some smart check can be
nice.

Regards

Pavel

> The idea would be that if you added a field that wasn't supposed to be
> copied, you'd have to add something to copyfuncs.c that said, e.g.
>
> /* NOTCOPIED: mymember */
>
> ...and the checking script would ignore things so flagged.
>
> --
> Robert Haas
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Fabien COELHO 2017-03-21 20:50:45 Re: [HACKERS] pgsql: Add missing support for new node fields
Previous Message Andrew Dunstan 2017-03-21 15:12:02 pgsql: Document btree_gin support for enums

Browse pgsql-hackers by date

  From Date Subject
Next Message Simon Riggs 2017-03-21 16:19:53 Re: Partitioned tables and relfilenode
Previous Message Peter Eisentraut 2017-03-21 16:13:19 Re: identity columns