On Sun, May 26, 2019 at 6:24 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Anybody have thoughts about a different way to approach it?
I mean, in an ideal world, I think we'd never call back out to
ProcessUtility() from within AlterTable(). That seems like a pretty
clear layering violation. I assume the reason we've never tried to do
better is a lack of round tuits and/or sufficient motivation.
In terms of what we'd do instead, I suppose we'd try to move as much
as possible inside the ALTER TABLE framework proper and have
everything call into that.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company