From: | Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Ole Gjerde <gjerde(at)icebox(dot)org> |
Cc: | Postgres Hackers List <hackers(at)postgreSQL(dot)org> |
Subject: | Re: [HACKERS] v6.5 release ToDo |
Date: | 1999-05-15 22:33:20 |
Message-ID: | 199905152233.SAA24810@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I believe also we have:
DROP TABLE/RENAME TABLE doesn't remove extended files, *.1, *.2
as an open item. Do you see these problems there?
> > > Prob #1:
> > > DROP TABLE <table> doesn't removed "extended" files.
> > > i.e., if you have 2GB db on Linux(i386), it will only delete the main
> > > file and not the .1, .2, etc table files.
>
> I have looked at this.
> I made it so it rolled over files at 1MB. My table ended up with 120
> segments, and my indexes had 3(Yes, it DOES work!).
> DROP TABLE removed ALL segments from the table, but only the main index
> segment.
>
> So it looks like removing the table itself is using mdunlink in md.c,
> while removing indexes uses FileNameUnlink() which only unlinks 1 file.
> As far as I can tell, calling FileNameUnlink() and mdunlink() is basically
> the same, except mdunlink() deletes any extra segments.
>
> I've done some testing and it seems to work. It also passes regression
> tests(except float8, geometry and rules, but that's normal).
>
> If this patch is right, this fixes all known multi-segment problems on
> Linux.
>
> Ole Gjerde
>
> Patch for index drop:
> --- src/backend/catalog/index.c 1999/05/10 00:44:55 1.71
> +++ src/backend/catalog/index.c 1999/05/15 06:42:27
> @@ -1187,7 +1187,7 @@
> */
> ReleaseRelationBuffers(userindexRelation);
>
> - if (FileNameUnlink(relpath(userindexRelation->rd_rel->relname.data)) < 0)
> + if (mdunlink(userindexRelation) != SM_SUCCESS)
> elog(ERROR, "amdestroyr: unlink: %m");
>
> index_close(userindexRelation);
>
>
>
--
Bruce Momjian | http://www.op.net/~candle
maillist(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
From | Date | Subject | |
---|---|---|---|
Next Message | Ole Gjerde | 1999-05-15 23:38:17 | Re: [HACKERS] v6.5 release ToDo |
Previous Message | Bruce Momjian | 1999-05-15 22:31:58 | Re: [HACKERS] v6.5 release ToDo |