Re: [BUGS] Patch to allow C extension modules to initialize/finish

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Joe Conway <mail(at)joeconway(dot)com>
Cc: David Fetter <david(at)fetter(dot)org>, rse(at)engelschall(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [BUGS] Patch to allow C extension modules to initialize/finish
Date: 2006-08-03 22:52:36
Message-ID: 27226.1154645556@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Joe Conway <mail(at)joeconway(dot)com> writes:
> Tom Lane wrote:
>> Also, if we do this we probably ought to remove the special-purpose
>> hack for preload_libraries to specify an init function --- it should
>> just happen by default. Any objections to simplifying that?

> The original idea of using the init function with preload_libraries was
> to eliminate library startup that was expensive and only needed once.
> Specifically in the case of libR (and presumably other libraries as
> well), the init time was much greater than the actual library load time.
> If it is removed from preload_libraries, then we'll pay that price for
> every backend startup, no?

No, my thought is that you'd rename PL/R's init function to PG_init, and
then it'd get called automagically without needing to assume that the DBA
remembers to specify it in preload_libraries. If there's a reason *not*
to do that then it'd be a strike against this whole proposal, methinks.

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2006-08-03 22:58:49 Re: BUG #2554: ILIKE operator works incorrectly
Previous Message Peter Eisentraut 2006-08-03 22:50:14 Re: BUG #2554: ILIKE operator works incorrectly

Browse pgsql-hackers by date

  From Date Subject
Next Message Zoltan Boszormenyi 2006-08-03 22:58:08 Re: GENERATED ... AS IDENTITY, Was: Re: Feature Freeze
Previous Message Bruce Momjian 2006-08-03 22:48:45 Re: pg_terminate_backend