diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2004-05-08 19:09:25 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2004-05-08 19:09:25 +0000 |
| commit | 4af3421161ce7847a019ec0799c898586574801f (patch) | |
| tree | 3ec8744681ef2d3e03b4479a677151a15a68b3f1 /src/backend/commands/sequence.c | |
| parent | 1fe11fad543fcaaf9ae7826f1234530a9d59c1b0 (diff) | |
| download | postgresql-4af3421161ce7847a019ec0799c898586574801f.tar.gz postgresql-4af3421161ce7847a019ec0799c898586574801f.zip | |
Get rid of rd_nblocks field in relcache entries. Turns out this was
costing us lots more to maintain than it was worth. On shared tables
it was of exactly zero benefit because we couldn't trust it to be
up to date. On temp tables it sometimes saved an lseek, but not often
enough to be worth getting excited about. And the real problem was that
we forced an lseek on every relcache flush in order to update the field.
So all in all it seems best to lose the complexity.
Diffstat (limited to 'src/backend/commands/sequence.c')
| -rw-r--r-- | src/backend/commands/sequence.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/backend/commands/sequence.c b/src/backend/commands/sequence.c index de4499124d5..0ff1b386b5b 100644 --- a/src/backend/commands/sequence.c +++ b/src/backend/commands/sequence.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/commands/sequence.c,v 1.109 2004/04/06 16:39:30 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/commands/sequence.c,v 1.110 2004/05/08 19:09:24 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -822,10 +822,6 @@ read_info(SeqTable elm, Relation rel, Buffer *buf) sequence_magic *sm; Form_pg_sequence seq; - if (rel->rd_nblocks > 1) - elog(ERROR, "invalid number of blocks in sequence \"%s\"", - RelationGetRelationName(rel)); - *buf = ReadBuffer(rel, 0); if (!BufferIsValid(*buf)) elog(ERROR, "ReadBuffer failed"); |