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.
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 |
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 |