Re: [PATCHES] WIP 2 interpreters for plperl

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCHES] WIP 2 interpreters for plperl
Date: 2006-11-13 17:16:16
Message-ID: 4558A860.5090006@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

I wrote:
>
> [moving to -hackers]
>
> I wrote:
>>
>>>
>>> I have made some progress with what I think is needed to have two
>>> interpreters for plperl. This is a lot harder than the pltcl case
>>> for two reasons: 1. there are no restrictions on having 2 tcl
>>> interpreters, and 2. tcl does not need to save and restore context
>>> as we have to do with perl. I think I have a conceptual siolution to
>>> these two problems, but what I have is currently segfaulting
>>> somewhat myteriously. Tracing a dynamically loaded library in a
>>> postgres backend with a debugger is less than fun, too. I am
>>> attaching what I currently have, liberally sprinkled with
>>> elog(NOTICE) calls as trace writes.
>>>
>>>
>>
>> With a little more perseverance I found the problem. The attached
>> patch passes regression. But it now needs plenty of eyeballs and
>> testing.
>>
>>
>
> Well, if anyone cast eyeballs over it they kept it secret from me :-(
>
>
> However, I have now tested the patch with the little script shown
> below and it seems to do the Right Thing (tm) in switching context and
> restoring it. So I think it can be applied to HEAD, along with an
> addition to the docs and a release note.
>
> Since this is a behaviour modification, do we want to apply it to the
> back branches? Doing so would certainly be possible, although it would
> be non-trivial.
>

I have committed this to HEAD at any rate, so that we can get some
buildfarm testing going.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message August Zajonc 2006-11-13 18:32:51 Re: Frequent Update Project: Design Overview ofHOTUpdates
Previous Message Enrico 2006-11-13 15:28:01 Re: EAN barcode datatype

Browse pgsql-patches by date

  From Date Subject
Next Message Pavan Deolasee 2006-11-14 19:01:22 Frequent Update - Heap Overflow Tuple (HOT) patch
Previous Message Dave Page 2006-11-13 11:16:25