From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Brent Verner <brent(at)rcfile(dot)org> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Problem reloading regression database |
Date: | 2002-01-25 01:14:49 |
Message-ID: | 200201250114.g0P1End23598@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
This has been saved for the 7.3 release:
http://candle.pha.pa.us/cgi-bin/pgpatches2
---------------------------------------------------------------------------
Brent Verner wrote:
> [2002-01-14 21:52] Tom Lane said:
> | Brent Verner <brent(at)rcfile(dot)org> writes:
> | > gotcha. No, I don't think anything inside that loop expects to
> | > persist across iterations. The attribute_buf is static to the
> | > file, and initialized in DoCopy.
> |
> | There is more to attribute_buf than meets the eye ;-)
> |
> | > What I ended up doing is switching to per-tuple-context prior to
> | > the input loop, then switching back to the (saved) query-context
> | > after exiting the loop. I followed ResetTupleExprContext back, and
> | > it doesn't seem to do anything that would require a switch per loop.
> | > Are there any problems this might cause that I'm not seeing with
> | > my test case?
> |
> | I really don't feel comfortable with running heap_insert or the
> | subsequent operations in a per-tuple context. Have you tried any
> | test cases that involve triggers or indexes?
>
> Yes. The attached patch appears to do the right thing with all
> indexes and triggers (RI) that I've tested. I'm still doing the
> MemoryContextSwitchTo() outside the main loop, and have added some
> more sanity checking for column name input.
>
> If anyone could test this (with non-critical data ;-) or otherwise
> give feedback, I'd appreciate it; especially if someone could test
> with a BEFORE INSERT trigger.
>
> cheers.
> brent
>
> --
> "Develop your talent, man, and leave the world something. Records are
> really gifts from people. To think that an artist would love you enough
> to share his music with anyone is a beautiful thing." -- Duane Allman
[ Attachment, skipping... ]
--
Bruce Momjian | http://candle.pha.pa.us
pgman(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 | Michael Devogelaere | 2002-01-25 01:16:15 | Re: PostgreSQL crashes with Qmail-SQL |
Previous Message | Tom Lane | 2002-01-25 01:14:44 | Re: PostgreSQL crashes with Qmail-SQL |