On Sunday, July 31, 2022 12:12 PM Michael Paquier <mich...@paquier.xyz> wrote: > > 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.
Thanks! Best regards, Hou Zhijie
