On Sun, May 26, 2019 at 6:24 PM Tom Lane <t...@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
