Thread: [HACKERS] Addition of pg_dump --no-publications

[HACKERS] Addition of pg_dump --no-publications

From
Michael Paquier
Date:
Hi all,

I imagine that pg_dump -s would be the basic operation that users
would do first before creating a subcription on a secondary node, but
what I find surprising is that publications are dumped by default. I
don't find confusing that those are actually included by default to be
consistent with the way subcriptions are handled, what I find
confusing is that there are no options to not dump them, and no
options to bypass their restore.

So, any opinions about having pg_dump/pg_restore --no-publications?
Thanks,
-- 
Michael



Re: [HACKERS] Addition of pg_dump --no-publications

From
Michael Paquier
Date:
On Thu, May 11, 2017 at 3:19 PM, Michael Paquier
<michael.paquier@gmail.com> wrote:
> I imagine that pg_dump -s would be the basic operation that users
> would do first before creating a subcription on a secondary node, but
> what I find surprising is that publications are dumped by default. I
> don't find confusing that those are actually included by default to be
> consistent with the way subcriptions are handled, what I find
> confusing is that there are no options to not dump them, and no
> options to bypass their restore.
>
> So, any opinions about having pg_dump/pg_restore --no-publications?

And that's really a boring patch, giving the attached.
-- 
Michael

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Attachment

Re: [HACKERS] Addition of pg_dump --no-publications

From
Peter Eisentraut
Date:
On 5/11/17 21:59, Michael Paquier wrote:
> On Thu, May 11, 2017 at 3:19 PM, Michael Paquier
> <michael.paquier@gmail.com> wrote:
>> I imagine that pg_dump -s would be the basic operation that users
>> would do first before creating a subcription on a secondary node, but
>> what I find surprising is that publications are dumped by default. I
>> don't find confusing that those are actually included by default to be
>> consistent with the way subcriptions are handled, what I find
>> confusing is that there are no options to not dump them, and no
>> options to bypass their restore.
>>
>> So, any opinions about having pg_dump/pg_restore --no-publications?
> 
> And that's really a boring patch, giving the attached.

committed

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



Re: [HACKERS] Addition of pg_dump --no-publications

From
David Fetter
Date:
On Fri, May 12, 2017 at 10:59:27AM +0900, Michael Paquier wrote:
> On Thu, May 11, 2017 at 3:19 PM, Michael Paquier
> <michael.paquier@gmail.com> wrote:
> > I imagine that pg_dump -s would be the basic operation that users
> > would do first before creating a subcription on a secondary node, but
> > what I find surprising is that publications are dumped by default. I
> > don't find confusing that those are actually included by default to be
> > consistent with the way subcriptions are handled, what I find
> > confusing is that there are no options to not dump them, and no
> > options to bypass their restore.
> >
> > So, any opinions about having pg_dump/pg_restore --no-publications?
> 
> And that's really a boring patch, giving the attached.

While it's consistent with surrounding code, I find the use of ints to
express what is in essence a boolean condition puzzling.  Any
insights?

Best,
David.
-- 
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter      XMPP: david(dot)fetter(at)gmail(dot)com

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate



Re: [HACKERS] Addition of pg_dump --no-publications

From
Tom Lane
Date:
David Fetter <david@fetter.org> writes:
> While it's consistent with surrounding code, I find the use of ints to
> express what is in essence a boolean condition puzzling.  Any
> insights?

IIRC, it's forced by the getopt_long API, particularly the way that
the long-options struct has to be declared.
        regards, tom lane



Re: [HACKERS] Addition of pg_dump --no-publications

From
Michael Paquier
Date:
On Fri, May 12, 2017 at 10:19 PM, Peter Eisentraut
<peter.eisentraut@2ndquadrant.com> wrote:
> On 5/11/17 21:59, Michael Paquier wrote:
>> On Thu, May 11, 2017 at 3:19 PM, Michael Paquier
>> <michael.paquier@gmail.com> wrote:
>>> I imagine that pg_dump -s would be the basic operation that users
>>> would do first before creating a subcription on a secondary node, but
>>> what I find surprising is that publications are dumped by default. I
>>> don't find confusing that those are actually included by default to be
>>> consistent with the way subcriptions are handled, what I find
>>> confusing is that there are no options to not dump them, and no
>>> options to bypass their restore.
>>>
>>> So, any opinions about having pg_dump/pg_restore --no-publications?
>>
>> And that's really a boring patch, giving the attached.
>
> committed

Thanks.
-- 
Michael