On Tue, Nov 15, 2022 at 11:42:34AM -0300, Ranier Vilela wrote: > I find it very difficult not to have some tuple to be updated, > once inside CopyStatistics and the branch cost can get in the way, > but I don't object with your solution.
The code assumes that it is a possibility. > Missed AddRoleMems? > Could you continue with CatalogTupleInsertWithInfo, what do you think? This one has been left out on purpose. I was tempting to use WithInfo() with a CatalogIndexState opened optionally but I got the impression that it makes the code a bit harder to follow and AddRoleMems() is already complex on its own. Most DDL patterns working on role would involve one role. More roles could be added of course in one shot, but the extra logic complexity did not look that appealing to me especially as some role updates are skipped. -- Michael
signature.asc
Description: PGP signature
