Skip site navigation (1) Skip section navigation (2)

Re: Why is vacuum_defer_cleanup_age PGC_USERSET?

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgreSQL(dot)org
Subject: Re: Why is vacuum_defer_cleanup_age PGC_USERSET?
Date: 2010-07-03 18:03:05
Message-ID: 1278180185.4151.9237.camel@ebony (view raw or flat)
Thread:
Lists: pgsql-hackers
On Sat, 2010-07-03 at 13:07 -0400, Tom Lane wrote:
> It seems to me it ought to be PGC_SIGHUP.  In the first place, the
> actually important value under normal circumstances is going to be
> that seen by autovac workers.  There is also some impact on HOT
> cleanup, but I can't see any sane argument why you don't want all
> backends to be applying the same value for that, since it's difficult
> or impossible to predict which session is actually going to HOT-clean
> any page.  In the second place, it seems completely foolish to let
> ordinary unprivileged users mess with the value.  Arguably an individual
> user could produce a denial of service on HS slaves by cutting his local
> value of vacuum_defer_cleanup_age and then vacuuming tables that queries
> on the slaves will look at.  To the extent that the variable does
> anything useful at all, it is the system-wide behavior that is
> important, so I see no use-case for changing it in individual sessions
> anyway.

Happy with that argument, so agreed.

> It also appears to me to be misclassified.  WAL_STANDBY_SERVERS should
> be the category for variables that you'd adjust on an HS slave, no?
> But this is something that has to be set on the master.  Possibly the
> best place for it is WAL_REPLICATION.

Those didn't exist when it was originally classified, thats all.

Yes, those categories sounds good.

-- 
 Simon Riggs           www.2ndQuadrant.com


In response to

pgsql-hackers by date

Next:From: Heikki LinnakangasDate: 2010-07-03 19:17:48
Subject: Re: Keepalive for max_standby_delay
Previous:From: Tom LaneDate: 2010-07-03 17:07:32
Subject: Why is vacuum_defer_cleanup_age PGC_USERSET?

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group