diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2000-06-18 22:44:35 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2000-06-18 22:44:35 +0000 |
| commit | 1ee26b776475155ad1fb00fa3ed0a93659ffadad (patch) | |
| tree | 1f2c7a59a1fdf3fe3eb62cf5044c5c6c21f77d12 /src/backend/utils/cache/relcache.c | |
| parent | 2c0edb3c8677831d836fc44eb58ebecb73f747af (diff) | |
| download | postgresql-1ee26b776475155ad1fb00fa3ed0a93659ffadad.tar.gz postgresql-1ee26b776475155ad1fb00fa3ed0a93659ffadad.zip | |
Reimplement nodeMaterial to use a temporary BufFile (or even memory, if the
materialized tupleset is small enough) instead of a temporary relation.
This was something I was thinking of doing anyway for performance, and Jan
says he needs it for TOAST because he doesn't want to cope with toasting
noname relations. With this change, the 'noname table' support in heap.c
is dead code, and I have accordingly removed it. Also clean up 'noname'
plan handling in planner --- nonames are either sort or materialize plans,
and it seems less confusing to handle them separately under those names.
Diffstat (limited to 'src/backend/utils/cache/relcache.c')
| -rw-r--r-- | src/backend/utils/cache/relcache.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c index 4fe3e35b1e8..93375a7c158 100644 --- a/src/backend/utils/cache/relcache.c +++ b/src/backend/utils/cache/relcache.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.101 2000/06/17 21:48:55 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.102 2000/06/18 22:44:17 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -1784,21 +1784,15 @@ RelationPurgeLocalRelation(bool xactCommitted) if (!xactCommitted) { - /* * remove the file if we abort. This is so that files for * tables created inside a transaction block get removed. */ - if (reln->rd_isnoname) + if (! reln->rd_unlinked) { - if (!(reln->rd_unlinked)) - { - smgrunlink(DEFAULT_SMGR, reln); - reln->rd_unlinked = TRUE; - } - } - else smgrunlink(DEFAULT_SMGR, reln); + reln->rd_unlinked = true; + } } if (!IsBootstrapProcessingMode()) |