Re: Why does load_external_function() return PGFunction? - Mailing list pgsql-hackers

From Andres Freund
Subject Re: Why does load_external_function() return PGFunction?
Date
Msg-id 20180207065458.sowsqipstl7kinhf@alap3.anarazel.de
Whole thread Raw
In response to Re: Why does load_external_function() return PGFunction?  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Why does load_external_function() return PGFunction?
List pgsql-hackers
On 2018-02-06 15:43:29 -0500, Tom Lane wrote:
> Andres Freund <andres@anarazel.de> writes:
> > We've several callers to load_external_function() that do not use the
> > returned value as a PGFunction. I'd vote for changing the return type to
> > void * and have fmgr.c cast it to PGFunction after verifying the
> > function's magic.
> 
> void* isn't necessarily compatible with function pointers --- there are
> platforms where they're physically different widths, though possibly
> you'd never get PG to run on such hardware anyway.

Fair point. Although we're relying on dlsym like infrastructure, which
returns just a void *.


> However, given that a cast is going to be necessary anyway, it seems
> like this is mostly useless churn...

Ok, can live with that too. Just though it was a bit weird...

Greetings,

Andres Freund


pgsql-hackers by date:

Previous
From: Pavel Stehule
Date:
Subject: Re: [HACKERS] proposal: schema variables
Next
From: Amit Khandekar
Date:
Subject: Re: Query running for very long time (server hanged) with parallel append