Re: New pg_dump patch, --no-stats flag, disables sending to statistics collector

From: Bryce Nesbitt <bryce2(at)obviously(dot)com>
To: Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, Jaime Casanova <jcasanov(at)systemguards(dot)com(dot)ec>
Subject: Re: New pg_dump patch, --no-stats flag, disables sending to statistics collector
Date: 2009-01-20 22:15:44
Message-ID: 49764D10.4000009@obviously.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Bruce Momjian wrote:
> Jaime Casanova wrote:
>
>> i haven't looked at the patch nor it's functional use... but from the
>> top of my head jumps a question: is there a reason to not make this
>> the default behaviour?
>>
> If this is a generally desired feature (and I question that), I think we
> need a more general solution.
>
I'm not a big fan of flags, preferring good defaults. But I was not
bold enough to suggest this as a new default, as someone would probably
want the opposite flag. If you're measuring total server load (rather
than analyzing an application), you may want to see pg_dump activity.

As for a "general" solution: one could add the ability to inject
arbitrary sql just prior to a dump run. That would let someone roll
their own by injecting "SET stats_block_level = false", or make any
other arbitrary settings changes.

Or one might slice the statistics collector by role or user (so your
'backup' role would keep a separate tally).

On the other hand, the flag's advantage is simplicity and directness.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message James Mansion 2009-01-20 22:40:09 Re: libpq WSACleanup is not needed
Previous Message Greg Smith 2009-01-20 22:13:53 Re: New pg_dump patch, --no-stats flag, disables sending to statistics collector

Browse pgsql-patches by date

  From Date Subject
Next Message Josh Berkus 2009-01-20 22:40:34 Re: New pg_dump patch, --no-stats flag, disables sending to statistics collector
Previous Message Greg Smith 2009-01-20 22:13:53 Re: New pg_dump patch, --no-stats flag, disables sending to statistics collector