diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2009-08-27 17:19:31 +0000 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2009-08-27 17:19:31 +0000 |
| commit | b78b4d5117b9277253b04b53838789b6ff3f01ed (patch) | |
| tree | c59b809ecf206550f2a12fed3363c9a7c5954317 /src/backend/access/common/reloptions.c | |
| parent | 593810f3cd7d4a5b502712daceb6a4ef6178c3c0 (diff) | |
| download | postgresql-b78b4d5117b9277253b04b53838789b6ff3f01ed.tar.gz postgresql-b78b4d5117b9277253b04b53838789b6ff3f01ed.zip | |
Fix handling of autovacuum reloptions.
In the original coding, setting a single reloption would cause default
values to be used for all the other reloptions. This is a problem
particularly for autovacuum reloptions.
Itagaki Takahiro
Diffstat (limited to 'src/backend/access/common/reloptions.c')
| -rw-r--r-- | src/backend/access/common/reloptions.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/backend/access/common/reloptions.c b/src/backend/access/common/reloptions.c index b970601b1c8..e1ce8596132 100644 --- a/src/backend/access/common/reloptions.c +++ b/src/backend/access/common/reloptions.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/access/common/reloptions.c,v 1.28 2009/06/11 14:48:53 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/access/common/reloptions.c,v 1.28.2.1 2009/08/27 17:19:31 alvherre Exp $ * *------------------------------------------------------------------------- */ @@ -108,7 +108,7 @@ static relopt_int intRelOpts[] = "Minimum number of tuple updates or deletes prior to vacuum", RELOPT_KIND_HEAP | RELOPT_KIND_TOAST }, - 50, 0, INT_MAX + -1, 0, INT_MAX }, { { @@ -116,7 +116,7 @@ static relopt_int intRelOpts[] = "Minimum number of tuple inserts, updates or deletes prior to analyze", RELOPT_KIND_HEAP | RELOPT_KIND_TOAST }, - 50, 0, INT_MAX + -1, 0, INT_MAX }, { { @@ -124,7 +124,7 @@ static relopt_int intRelOpts[] = "Vacuum cost delay in milliseconds, for autovacuum", RELOPT_KIND_HEAP | RELOPT_KIND_TOAST }, - 20, 0, 100 + -1, 0, 100 }, { { @@ -132,7 +132,7 @@ static relopt_int intRelOpts[] = "Vacuum cost amount available before napping, for autovacuum", RELOPT_KIND_HEAP | RELOPT_KIND_TOAST }, - 200, 1, 10000 + -1, 1, 10000 }, { { @@ -140,7 +140,7 @@ static relopt_int intRelOpts[] = "Minimum age at which VACUUM should freeze a table row, for autovacuum", RELOPT_KIND_HEAP | RELOPT_KIND_TOAST }, - 100000000, 0, 1000000000 + -1, 0, 1000000000 }, { { @@ -148,14 +148,14 @@ static relopt_int intRelOpts[] = "Age at which to autovacuum a table to prevent transaction ID wraparound", RELOPT_KIND_HEAP | RELOPT_KIND_TOAST }, - 200000000, 100000000, 2000000000 + -1, 100000000, 2000000000 }, { { "autovacuum_freeze_table_age", "Age at which VACUUM should perform a full table sweep to replace old Xid values with FrozenXID", RELOPT_KIND_HEAP | RELOPT_KIND_TOAST - }, 150000000, 0, 2000000000 + }, -1, 0, 2000000000 }, /* list terminator */ {{NULL}} @@ -169,7 +169,7 @@ static relopt_real realRelOpts[] = "Number of tuple updates or deletes prior to vacuum as a fraction of reltuples", RELOPT_KIND_HEAP | RELOPT_KIND_TOAST }, - 0.2, 0.0, 100.0 + -1, 0.0, 100.0 }, { { @@ -177,7 +177,7 @@ static relopt_real realRelOpts[] = "Number of tuple inserts, updates or deletes prior to analyze as a fraction of reltuples", RELOPT_KIND_HEAP | RELOPT_KIND_TOAST }, - 0.1, 0.0, 100.0 + -1, 0.0, 100.0 }, /* list terminator */ {{NULL}} |