Re: Add on_perl_init and proper destruction to plperl [PATCH]

Поиск
Список
Период
Сортировка
От David E. Wheeler
Тема Re: Add on_perl_init and proper destruction to plperl [PATCH]
Дата
Msg-id C8ECC818-B9CB-4364-941E-A2A6DE3C6163@kineticode.com
обсуждение исходный текст
Ответ на Re: Add on_perl_init and proper destruction to plperl [PATCH]  (Tim Bunce <Tim.Bunce@pobox.com>)
Ответы Re: Add on_perl_init and proper destruction to plperl [PATCH]  (Robert Haas <robertmhaas@gmail.com>)
Re: Add on_perl_init and proper destruction to plperl [PATCH]  (Tim Bunce <Tim.Bunce@pobox.com>)
Список pgsql-hackers
On Jan 27, 2010, at 1:27 PM, Tim Bunce wrote:

> Okay. I could change the callback code to ignore calls if
> proc_exit_inprogress is false. So an abnormal shutdown via exit()
> wouldn't involve plperl at all. (Alternatively I could use use
> on_proc_exit() instead of atexit() to register the callback.)

Given Tom's hesitace about atexit(), perhaps that would be best.

> Neither of those relate to the actions of perl source code.
> To address that, instead of calling perl_destruct() to perform a
> complete destruction I could just execute END blocks and object
> destructors. That would avoid executing any system-level actions.

Does perl_destruct() execute system-level actions, then? If so, then it seems prudent to either audit such actions or,
asyou say, call destructors directly. 

> Do you have any examples of how a user could write code in a plperl END
> block that would interact with the rest of the backend?

I appreciate you taking the time to look at ways to address the issues Tom has raised, Tim. Good on you.

There is so much benefit to this level of interaction, as shown by the success of mod_perl and other forking
environmentsthat support pre-loading code, that I think it'd be extremely valuable to get these features in 9.0. They
reallyallow Perl to be a first-class PL in a way that it wasn't before. 

So if there is no way other than SPI for Perl code to interact with the backend, and system-level actions in Perl
itselfare disabled, it seems to me that the major issues are addressed. Am I wrong, Tom? 

Best,

David

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Greg Smith
Дата:
Сообщение: Re: Clustering Docs WAS: Mammoth in Core?
Следующее
От: Robert Haas
Дата:
Сообщение: Re: Add on_perl_init and proper destruction to plperl [PATCH]