pg_dump not dumping the run_as_owner setting from version 16?
Hi,
I as far as I can tell, pg_dump does not dup the ‘run_as_owner` setting for a subscription.
Should it? Should I submit a patch? It seems pretty trivial to fix if anyone else is working on it.
Sent from Mail for Windows
Further to this: it seems that `Alter Subscription X Set(Run_As_Owner=True);` has no influence on the `subrunasowner` column of pg_subscriptions.
Sent from Mail for Windows
From: Philip Warner
Sent: Friday, 27 October 2023 3:26 PM
To: pgsql-hackers@lists.postgresql.org
Subject: pg_dump not dumping the run_as_owner setting from version 16?
Hi,
I as far as I can tell, pg_dump does not dup the ‘run_as_owner` setting for a subscription.
Should it? Should I submit a patch? It seems pretty trivial to fix if anyone else is working on it.
Sent from Mail for Windows
On Fri, 2023-10-27 at 18:05 +1100, Philip Warner wrote:
I as far as I can tell, pg_dump does not dup the ‘run_as_owner` setting for a subscription.
Should it? Should I submit a patch? It seems pretty trivial to fix if anyone else is working on it.
Yes, it certainly should. That is an omission in 482675987b.
Go ahead and write a fix!
Further to this: it seems that `Alter Subscription X Set(Run_As_Owner=True);`
has no influence on the `subrunasowner` column of pg_subscriptions.
This seems to have been fixed in f062cddafe.
Yours,
Laurenz Albe
I as far as I can tell, pg_dump does not dup the ‘run_as_owner` setting for a subscription.
Should it? Should I submit a patch? It seems pretty trivial to fix if anyone else is working on it.
Yes, it certainly should. That is an omission in 482675987b.
Go ahead and write a fix!
Please find attached a patch for pg_dump to honour the setting of `run_as_owner`; I believe that effective pre-16 behavious was to run as owner, so I have set the flag to ‘t’ for pre-16 versions. Please let me know if you would prefer the opposite.
Further to this: it seems that `Alter Subscription X Set(Run_As_Owner=True);`
has no influence on the `subrunasowner` column of pg_subscriptions.This seems to have been fixed in f062cddafe.
Yes, I can confirm that in the current head `pg_subscriptions` reflects the setting correctly.
...patch actually attached this time...
I as far as I can tell, pg_dump does not dup the ‘run_as_owner` setting for a subscription.
Should it? Should I submit a patch? It seems pretty trivial to fix if anyone else is working on it.
Yes, it certainly should. That is an omission in 482675987b.
Go ahead and write a fix!
Please find attached a patch for pg_dump to honour the setting of `run_as_owner`; I believe that effective pre-16 behavious was to run as owner, so I have set the flag to ‘t’ for pre-16 versions. Please let me know if you would prefer the opposite.
Further to this: it seems that `Alter Subscription X Set(Run_As_Owner=True);`
has no influence on the `subrunasowner` column of pg_subscriptions.This seems to have been fixed in f062cddafe.
Yes, I can confirm that in the current head `pg_subscriptions` reflects the setting correctly.
Attachments:
pg_dump-run_as_owner.patchapplication/octet-stream; name=pg_dump-run_as_owner.patchDownload+9-0
Philip Warner <pjw@rhyme.com.au> writes:
Please find attached a patch for pg_dump to honour the setting of `run_as_owner`; I believe that effective pre-16 behavious was to run as owner, so I have set the flag to ‘t’ for pre-16 versions. Please let me know if you would prefer the opposite.
I think that's the correct choice. Fix pushed, thanks.
regards, tom lane