NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit52985e4

committed

Revert "Increase the default vacuum_cost_limit from 200 to 2000"

This reverts commitbd09503.Per discussion, it seems like what we should do instead is toreduce the default value of autovacuum_vacuum_cost_delay by thesame factor. That's functionally equivalent as long as theplatform can accurately service the smaller delay request, whichshould be true on anything released in the last 10 years or more.And smaller, more-closely-spaced delays are better in terms ofproviding a steady I/O load.Discussion:https://postgr.es/m/28720.1552101086@sss.pgh.pa.us

1 parentcaf626b commit52985e4Copy full SHA for 52985e4

File tree

4 files changed

-4

lines changed

doc/src/sgml
- config.sgml
src/backend/utils
- init
  - globals.c
- misc
  - guc.c
  - postgresql.conf.sample

4 files changed

-4

lines changed

`‎doc/src/sgml/config.sgml`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1932,7 +1932,7 @@ include_dir 'conf.d'`
`1932`	`1932`	`<listitem>`
`1933`	`1933`	`<para>`
`1934`	`1934`	`The accumulated cost that will cause the vacuuming process to sleep.`
`1935`		`- The default value is2000.`
	`1935`	`+ The default value is200.`
`1936`	`1936`	`</para>`
`1937`	`1937`	`</listitem>`
`1938`	`1938`	`</varlistentry>`

`‎src/backend/utils/init/globals.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,7 @@ intMaxBackends = 0;`
`137`	`137`	`intVacuumCostPageHit=1;/* GUC parameters for vacuum */`
`138`	`138`	`intVacuumCostPageMiss=10;`
`139`	`139`	`intVacuumCostPageDirty=20;`
`140`		`-intVacuumCostLimit=2000;`
	`140`	`+intVacuumCostLimit=200;`
`141`	`141`	`doubleVacuumCostDelay=0;`
`142`	`142`
`143`	`143`	`intVacuumPageHit=0;`

`‎src/backend/utils/misc/guc.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2265,7 +2265,7 @@ static struct config_int ConfigureNamesInt[] =`
`2265`	`2265`	`NULL`
`2266`	`2266`	`},`
`2267`	`2267`	`&VacuumCostLimit,`
`2268`		`-2000,1,10000,`
	`2268`	`+200,1,10000,`
`2269`	`2269`	`NULL,NULL,NULL`
`2270`	`2270`	`},`
`2271`	`2271`

`‎src/backend/utils/misc/postgresql.conf.sample`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -159,7 +159,7 @@`
`159`	`159`	`#vacuum_cost_page_hit = 1# 0-10000 credits`
`160`	`160`	`#vacuum_cost_page_miss = 10# 0-10000 credits`
`161`	`161`	`#vacuum_cost_page_dirty = 20# 0-10000 credits`
`162`		`-#vacuum_cost_limit =2000# 1-10000 credits`
	`162`	`+#vacuum_cost_limit =200# 1-10000 credits`
`163`	`163`
`164`	`164`	`# - Background Writer -`
`165`	`165`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit52985e4

File tree

4 files changed

4 files changed

`‎doc/src/sgml/config.sgml`

`‎src/backend/utils/init/globals.c`

`‎src/backend/utils/misc/guc.c`

`‎src/backend/utils/misc/postgresql.conf.sample`

0 commit comments