On Sat, Jan 07, 2023 at 07:37:49AM +0100, Peter Eisentraut wrote:
> The generation script already has a way to identify location fields, by ($t
> eq 'int' && $f =~ 'location$'), so you could use that as well.
I recall that some of the nodes may need renames to map with this
choice. That could be just one patch on top of the actual feature.
> I'm concerned about the large number of additional field annotations this
> adds. We have been careful so far to document the use of each attribute,
> e.g., *why* does a field not need to be copied etc. This patch adds dozens
> and dozens of annotations without any explanation at all. Now, the code
> this replaces also has no documentation, but maybe this is the time to add
> some.
In most cases, the addition of the node marker would be enough to
self-explain why they are included, but there is a trend for a lot of
the nodes when it comes to collations and typmods where we don't want
to see these in the jumbling calculations. I'll look at providing
more info for all that. (Note: I'm out for now.)
--
Michael
