| From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
|---|---|
| To: | Andres Freund <andres(at)anarazel(dot)de> |
| Cc: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>, Alvaro Herrera <alvherre(at)commandprompt(dot)com>, pgsql-bugs <pgsql-bugs(at)postgresql(dot)org>, Hans van Kranenburg <hans(dot)van(dot)kranenburg(at)mendix(dot)com> |
| Subject: | Re: BUG #5566: High levels of savepoint nesting trigger stack overflow in AssignTransactionId |
| Date: | 2010-07-22 21:01:05 |
| Message-ID: | AANLkTinwgGouvph5kgnx0J=sHxxm7ZUPspGbHtOOuK9p@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
On Mon, Jul 19, 2010 at 4:35 PM, Andres Freund <andres(at)anarazel(dot)de> wrote:
>> Well. I got that far. But why is that something worthy of support?
>> For one I have a hard time imaging a sensible use case, for another doing
>> anything in that deeply nested transactions seems to gets really slow (the
>> chain of transactions gets walked at some places for one thing, there seem
>> to be others).
>>
>> If want I can write a patch for that as well, seems to be trivial enough.
> Updated patch attached.
Considering that this is a crasher, I think we'll need to back-patch
this. The proposed patch applies only as far back as 8.3, due to the
lazy XID assignment changes in that version, but it looks like the bug
exists all the way back to 8.0. It looks like only minor adjustments
are required for the older branches, though. 7.4 is not affected, as
it does not have subtransactions.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2010-07-22 22:29:47 | Re: BUG #5566: High levels of savepoint nesting trigger stack overflow in AssignTransactionId |
| Previous Message | Alex Hunsaker | 2010-07-22 19:02:55 | Re: BUG #5567: will not install |