On Tue, Sep 07, 2021 at 10:57:02AM +0200, Peter Eisentraut wrote: > On 02.09.21 20:53, Jacob Champion wrote: > >>0004-Make-node-output-prefix-match-node-structure-name.patch > >> > >>Some nodes' output/read functions use a label that is slightly different > >>from their node name, e.g., "NOTIFY" instead of "NOTIFYSTMT". This > >>cleans that up so that an automated approach doesn't have to deal with > >>these special cases. > > > >Is there any concern about the added serialization length, or is that > >trivial in practice? The one that particularly caught my eye is > >RANGETBLENTRY, which was previously RTE. But I'm not very well-versed > >in all the places these strings can be generated and stored. > > These are just matters of taste. Let's wait a bit more to see if anyone is > concerned.
I am not concerned about changing the serialization length this much. The format is already quite verbose, and this change is small relative to that existing verbosity.
