RE: Collect ObjectAddress for ATTACH DETACH PARTITION to use in event trigger - Mailing list pgsql-hackers

From houzj.fnst@fujitsu.com
Subject RE: Collect ObjectAddress for ATTACH DETACH PARTITION to use in event trigger
Date
Msg-id OS0PR01MB5716DEAF92CB2DF114C0E68694949@OS0PR01MB5716.jpnprd01.prod.outlook.com
Whole thread Raw
In response to Re: Collect ObjectAddress for ATTACH DETACH PARTITION to use in event trigger  (Michael Paquier <michael@paquier.xyz>)
Responses Re: Collect ObjectAddress for ATTACH DETACH PARTITION to use in event trigger
List pgsql-hackers
On Monday, July 25, 2022 6:26 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Mon, Jul 25, 2022 at 09:25:07AM +0000, houzj.fnst@fujitsu.com wrote:
> > 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.
>
> @@ -300,6 +300,18 @@ get_altertable_subcmdinfo(PG_FUNCTION_ARGS)
> [ ... ]
>     default:
>         strtype = "unrecognized";
>         break;
>
> Removing the "default" clause would help here as we would get compiler
> warnings if there is anything missing.  One way to do things is to set strtype to
> NULL before going through the switch and have a failsafe as some commands
> are internal so they may not be worth adding to the output.

Thanks for the suggestion. I have removed the default and found some missed
subcommands in 0003 patch. Attach the new version patch here
(The 0001 and 0002 is unchanged).

Best regards,
Hou zj



Attachment

pgsql-hackers by date:

Previous
From: Pierre
Date:
Subject: Re: log_line_prefix: make it possible to add the search_path
Next
From: Justin Pryzby
Date:
Subject: Re: Restructure ALTER TABLE notes to clarify table rewrites and verification scans