Re: [DOCS] [HACKERS] bgwriter_lru_multiplier blurbs inconsistent

2008-03-06 Thread Bruce Momjian
Tom Lane wrote:
> Greg Smith <[EMAIL PROTECTED]> writes:
> > On Sun, 20 Jan 2008, Tom Lane wrote:
> >> I think the main problem is the qualifying clause up front in a place
> >> of prominence.  Here's a V3 try
> 
> > That one looks good to me.  These are small details but better to get it 
> > right now.
> 
> OK, committed.  Back to Alvaro's original concern: is the short
> description in guc.c all right, or can we improve that?

I have tried to improve the GUC description for
"bgwriter_lru_multiplier";  applied to CVS HEAD.

-- 
  Bruce Momjian  <[EMAIL PROTECTED]>http://momjian.us
  EnterpriseDB http://postgres.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +
Index: src/backend/utils/misc/guc.c
===
RCS file: /cvsroot/pgsql/src/backend/utils/misc/guc.c,v
retrieving revision 1.432
diff -c -c -r1.432 guc.c
*** src/backend/utils/misc/guc.c	30 Jan 2008 18:35:55 -	1.432
--- src/backend/utils/misc/guc.c	6 Mar 2008 16:22:44 -
***
*** 1841,1847 
  
  	{
  		{"bgwriter_lru_multiplier", PGC_SIGHUP, RESOURCES,
! 			gettext_noop("Background writer multiplier on average buffers to scan per round."),
  			NULL
  		},
  		&bgwriter_lru_multiplier,
--- 1841,1847 
  
  	{
  		{"bgwriter_lru_multiplier", PGC_SIGHUP, RESOURCES,
! 			gettext_noop("Multiple of the average buffer usage to free per round."),
  			NULL
  		},
  		&bgwriter_lru_multiplier,

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://mail.postgresql.org/mj/mj_wwwusr?domain=postgresql.org&extra=pgsql-hackers


Re: [HACKERS] bgwriter_lru_multiplier blurbs inconsistent

2008-01-20 Thread Tom Lane
Greg Smith <[EMAIL PROTECTED]> writes:
> On Sun, 20 Jan 2008, Tom Lane wrote:
>> I think the main problem is the qualifying clause up front in a place
>> of prominence.  Here's a V3 try

> That one looks good to me.  These are small details but better to get it 
> right now.

OK, committed.  Back to Alvaro's original concern: is the short
description in guc.c all right, or can we improve that?

regards, tom lane

---(end of broadcast)---
TIP 3: Have you checked our extensive FAQ?

   http://www.postgresql.org/docs/faq


Re: [HACKERS] bgwriter_lru_multiplier blurbs inconsistent

2008-01-20 Thread Greg Smith

On Sun, 20 Jan 2008, Tom Lane wrote:


I think the main problem is the qualifying clause up front in a place
of prominence.  Here's a V3 try


That one looks good to me.  These are small details but better to get it 
right now.


--
* Greg Smith [EMAIL PROTECTED] http://www.gregsmith.com Baltimore, MD

---(end of broadcast)---
TIP 6: explain analyze is your friend


Re: [HACKERS] bgwriter_lru_multiplier blurbs inconsistent

2008-01-20 Thread Tom Lane
Greg Smith <[EMAIL PROTECTED]> writes:
> There is nothing incorrect here, it's just not as clear as it could be. 
> Here's a V2 that tries to clear that up:

> Unless limited by bgwriter_lru_maxpages, the number of dirty 
> buffers written in each round is based on the number of new buffers that 
> have been needed by server processes during recent rounds.  The recent 
> need is multiplied by bgwriter_lru_multiplier to arrive at the 
> estimate of the number of buffers that will be needed during the next 
> round.  Buffers are written to meet that need if there aren't enough 
> reusable ones found while scanning.

I think the main problem is the qualifying clause up front in a place
of prominence.  Here's a V3 try:

The number of dirty buffers written in each round is based on the number
of new buffers that have been needed by server processes during recent
rounds.  The average recent need is multiplied by
bgwriter_lru_multiplier to arrive at the estimate of the
number of buffers that will be needed during the next round.  Dirty
buffers are written until there are that many clean, reusable buffers
available.  However, no more than bgwriter_lru_maxpages
buffers will be written per round.

regards, tom lane

---(end of broadcast)---
TIP 2: Don't 'kill -9' the postmaster


Re: [HACKERS] bgwriter_lru_multiplier blurbs inconsistent

2008-01-20 Thread Greg Smith

On Sun, 20 Jan 2008, Alvaro Herrera wrote:


Is the bgwriter_lru_multiplier parameter a limit on the number to scan
or to write?  GUC and docs seem to contradict one another.


It adjusts the target for how many clean buffers it wants to either find 
or create.  This always increases the number of buffers scanned, and 
that's what the GUC description says.


Since a clean buffer can either be a) a reusable candidate found by 
scanning or b) a buffer that is written, on average it's adjusting up the 
number of writes as well.  But it's not guaranteed to--you could have a 
case where it just found all the buffers it needed and never wrote a 
single one.



Unless limited by bgwriter_lru_maxpages, the number
of dirty buffers written in each round is determined by reference
to the number of new buffers that have been needed by server
processes during recent rounds.  This number is multiplied by
bgwriter_lru_multiplier to arrive at the estimate
of the number of buffers that will be needed during the next round.


There is nothing incorrect here, it's just not as clear as it could be. 
Here's a V2 that tries to clear that up:


Unless limited by bgwriter_lru_maxpages, the number of dirty 
buffers written in each round is based on the number of new buffers that 
have been needed by server processes during recent rounds.  The recent 
need is multiplied by bgwriter_lru_multiplier to arrive at the 
estimate of the number of buffers that will be needed during the next 
round.  Buffers are written to meet that need if there aren't enough 
reusable ones found while scanning.


--
* Greg Smith [EMAIL PROTECTED] http://www.gregsmith.com Baltimore, MD

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
  choose an index scan if your joining column's datatypes do not
  match


[HACKERS] bgwriter_lru_multiplier blurbs inconsistent

2008-01-20 Thread Alvaro Herrera
Is the bgwriter_lru_multiplier parameter a limit on the number to scan
or to write?  GUC and docs seem to contradict one another.  GUC says

#: utils/misc/guc.c:1834
#, fuzzy
msgid "Background writer multiplier on average buffers to scan per round."

The docs say

 Unless limited by bgwriter_lru_maxpages, the number
 of dirty buffers written in each round is determined by reference
 to the number of new buffers that have been needed by server
 processes during recent rounds.  This number is multiplied by
 bgwriter_lru_multiplier to arrive at the estimate
 of the number of buffers that will be needed during the next round.


Which one is correct?  Do we need a correction of either?

-- 
Alvaro Herrerahttp://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.

---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
   subscribe-nomail command to [EMAIL PROTECTED] so that your
   message can get through to the mailing list cleanly