John Naylor <jcnay...@gmail.com> writes:
> Looking at convert_oid2name.patch again, I see this:

> +                     elsif ($catname eq 'pg_am')
> +                     {
> +                             $values{aggfnoid}     = 
> lookup_procname($values{aggfnoid});
> +                     }

> aggfnoid is in pg_aggregate, and pg_am already had a regproc lookup.
> Do you remember the intent here?

Ugh, copy-and-pasteo.  I intended to have it lookup pg_am.amhandler, but
must have missed changing the field name after copying code from the
pg_aggregate stanza.  Seems to have been unnecessary anyway, since all the
entries in the column are already symbolic.

                        regards, tom lane

Reply via email to