Re: plperl and inline functions -- first draft

On Nov 18, 2009, at 5:46 AM, Andrew Dunstan wrote:

>
>
> Joshua Tolley wrote:
>> +     plperl_call_data *save_call_data = current_call_data;
>> +     bool        oldcontext = trusted_context;
>> + +     if (SPI_connect() != SPI_OK_CONNECT)
>> +         elog(ERROR, "could not connect to SPI manager");
>>
> ...
>> +     current_call_data = (plperl_call_data *) palloc0(sizeof(plperl_call_data));
>> +     current_call_data->fcinfo = &fake_fcinfo;
>> +     current_call_data->prodesc = &desc;
>>
>
> I don't think this is done in the right order. If it is then this comment in plperl_func_handler is wrong (as well as
containinga typo): 
>
>   /*
>    * Create the call_data beforing connecting to SPI, so that it is not
>    * allocated in the SPI memory context
>    */
>

Yes, current_call_data can't be allocate in the SPI memory context, since it's used to extract the result after
SPI_finishis called, although it doesn't lead to problems here since no result is returned. Anyway, I'd move
SPI_connectafter the current_call_data initialization. 

I also noticed that no error context is set in the inline handler, not sure whether it really useful except for the
sakeof consistency, but in case it is - here is the patch: 


--
Alexey Klyukin                    http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc