On Sat, Jul 30, 2022 at 01:13:52PM +0000, houzj.f...@fujitsu.com wrote: > I am not against returning the objaddr for cases related to RLS and RelOption. > But just to confirm, do you have a use case to use the returned > address(relation itself) > for RLS or RelOptions in event trigger ? I asked this because when I tried to > deparse the subcommand of ALTER TABLE. It seems enough to use the information > inside the parse tree to deparse the RLS and RelOptions related subcommands.
You are right here, there is little point in returning the relation itself. I have removed these modifications, added a couple of extra commands for some extra coverage, and applied all that. I have finished by splitting the extension of test_ddl_deparse/ and the addition of ObjectAddress for the attach/detach into their own commit, mainly for clarity. -- Michael
signature.asc
Description: PGP signature
