2014/1/12 Marko Tiikkaja <ma...@joh.to> > On 1/12/14, 5:33 PM, I wrote: > >> On 1/9/14, 11:41 PM, Pavel Stehule wrote: >> >>> There are two basic questions: >>> >>> b) will we support same API still - a reference on plugin_info in exec >>> state is a issue - described in patch. >>> >> >> Pardon my ignorance, but why does the plugin_info have to be in the >> executor state? If we're going to change the API, can't we pass it >> directly to the callback function? >> > > Oh, I think I'm being stupid -- we'd only have to do what *if* we don't > want to change the API? Then my vote is for breaking the API. >
yes. It is my vote too. It is trouble - but support same API is really ugly - on second hand - there are only few plpgsql plugins - and every plugin needs recompilation for new mayor version and fixing will be easy. Regards Pavel Stehule > > > Regards, > Marko Tiikkaja >
