Re: [GENERAL] [PATCHES] A way to let Vacuum warn if FSM settings are low. [final?]

2005-04-07 Thread Tom Lane
Bruce Momjian pgman@candle.pha.pa.us writes: I never heard any discussion on whether this should be backpatched to 8.0.X. Should it? I'm not inclined to throw it in at the last minute, as it's not been through any testing and I'm not sure the behavior has really been agreed on anyway. (The

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low. [final?]

2005-03-13 Thread Bruce Momjian
Ron Mayer wrote: Bruce Momjian wrote: You didn't like server_min_messages = 'notify'? I merely don't have a feeling for how much additional stuff verbose would be putting in the log files. You will probably see the creation of indexes and sequences, like this: test=

Re: [GENERAL] [PATCHES] A way to let Vacuum warn if FSM settings are low. [final?]

2005-03-13 Thread Tom Lane
Bruce Momjian pgman@candle.pha.pa.us writes: Ron Mayer wrote: My reasoning why I thought the log file was more useful was that only an admin with access to the log files could really do anything about the message anyway. The log file is useful, but I think showing the VACUUM user is _more_

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low. [final?]

2005-03-12 Thread Ron Mayer
Well, I was really hoping something would end up in the log file. The situation is that our clients - sometimes not that computer savvy - go perhaps a year without us being involved (unless the log monitoring scripts show something abnormal; or if the system breaks). The primary motivation for

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low. [final?]

2005-03-12 Thread Bruce Momjian
Ron Mayer wrote: Well, I was really hoping something would end up in the log file. The situation is that our clients - sometimes not that computer savvy - go perhaps a year without us being involved (unless the log monitoring scripts show something abnormal; or if the system breaks). The

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low. [final?]

2005-03-11 Thread Bruce Momjian
I have applied your patch with minor modifications. Applied version attached. I think the pages message: INFO: free space map: 44 relations, 28 pages stored; 704 total pages used DETAIL: FSM size: 1000 relations + 2 pages = 182 kB shared memory. should remain

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low. [final?]

2005-02-28 Thread Bruce Momjian
[ Previous version removed.] Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it.

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-27 Thread Simon Riggs
On Fri, 2005-02-25 at 16:48 -0800, Ron Mayer wrote: Getting closer? For me, yes. I agree with Bruce's comment on the use of the word needed, and I think your change reads better now. The not-warnings seem a little wordy for me, but they happen when and how I would hope for. So, for me, it

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-25 Thread Ron Mayer
On Fri, 25 Feb 2005, Bruce Momjian wrote: Tom Lane wrote: Ron Mayer [EMAIL PROTECTED] writes: Should the relation overflow be a WARNING or a LOG? ... I'd go for making them both LOG, I think. More consistent. Can we also update this wording: INFO: free space map: 52 relations, 61

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-24 Thread Christopher Kings-Lynne
I find this tiny (9-line) patch useful to help my clients know when FSM settings may need updating. Some of the more frequently asked questions here are in regards to FSM settings. One hint I've seen is to run vacuum verbose;. At the end of thousands of lines of INFO and DETAIL messages

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-24 Thread Simon Riggs
On Wed, 2005-02-23 at 19:31 -0500, Tom Lane wrote: Ron Mayer [EMAIL PROTECTED] writes: +if (needed MaxFSMPages) +ereport(WARNING, +(errmsg(max_fsm_pages(%d) is smaller than total pages needed(%.0f), + MaxFSMPages, needed))); An unconditional

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-24 Thread Ron Mayer
Thanks everyone for the feedback on my patch. Objections I've heard (both online and in email) included: * WARNING is too strong for possibly OK behavior * It's similar to checkpoints occuring too frequently... consider increasing...checkpoint_segments which is a LOG

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-24 Thread Ron Mayer
On Thu, 24 Feb 2005, Tom Lane wrote: I preferred Simon's idea of not trying to produce a warning for pages when we've detected relation overflow. Sounds good. I'll make that update. Should the relation overflow be a WARNING or a LOG? It sounds like if you have that problem it's almost

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-24 Thread Tom Lane
Ron Mayer [EMAIL PROTECTED] writes: Should the relation overflow be a WARNING or a LOG? It sounds like if you have that problem it's almost certainly a problem, right? I'd go for making them both LOG, I think. More consistent. regards, tom lane

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-24 Thread Ron Mayer
On Thu, 24 Feb 2005, Ron Mayer wrote: Should the relation overflow be a WARNING or a LOG? It sounds like if you have that problem it's almost certainly a problem, right? And while I'm at it... what's the convention for INFOs vs LOGs? The checkpoint...too frequent seemed similar, and is a LOG.

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-24 Thread Ron Mayer
On Thu, 24 Feb 2005, Tom Lane wrote: I'd go for making them both LOG, I think. More consistent. Ok, here's another try :) With a couple more questions... 1. If I read Simon's email correctly, it implied that he wanted to see the free space map message for a VACUUM even when VERBOSE is

[PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-23 Thread Ron Mayer
Short summary: I find this tiny (9-line) patch useful to help my clients know when FSM settings may need updating. Some of the more frequently asked questions here are in regards to FSM settings. One hint I've seen is to run vacuum verbose;. At the end of thousands of lines of INFO and

Re: [PATCHES] A way to let Vacuum warn if FSM settings are low.

2005-02-23 Thread Tom Lane
Ron Mayer [EMAIL PROTECTED] writes: +if (needed MaxFSMPages) +ereport(WARNING, +(errmsg(max_fsm_pages(%d) is smaller than total pages needed(%.0f), + MaxFSMPages, needed))); An unconditional WARNING seems a bit strong to me for a case that is not