On Saturday, July 23, 2022 6:58 PM Michael Paquier <mich...@paquier.xyz> wrote: > > On Sat, Jul 23, 2022 at 05:44:28PM +0900, Michael Paquier wrote: > > Changing get_altertable_subcmdtypes() to return a set of rows made of > > (subcommand, object description) is what I actually meant upthread as > > it feels natural given a CollectedCommand in input, and as > > pg_event_trigger_ddl_commands() only gives access to a set of > > CollectedCommands. This is also a test module so there is no issue in > > changing the existing function definitions. > > > > But your point would be to have a new function that takes in input a > > CollectedATSubcmd, returning back the object address or its > > description? How would you make sure that a subcommand maps to a > > correct object address? > > FWIW, I was thinking about something among the lines of 0002 on top of Hou's > patch.
Thanks for the patches. The patches look good to me. BTW, while reviewing it, I found there are some more subcommands that the get_altertable_subcmdtypes() doesn't handle(e.g., ADD/DROP/SET IDENTITY and re ADD STAT). Shall we fix them all while on it ? Attach a minor patch to fix those which is based on the v2 patch set. Best regards, Hou zj
v2-0003-Add-support-for-some-missed-commands-in-test_ddl_dep_patch
Description: v2-0003-Add-support-for-some-missed-commands-in-test_ddl_dep_patch