Re: cache invalidation for PL/pgsql functions - Mailing list pgsql-hackers

From Robert Haas
Subject Re: cache invalidation for PL/pgsql functions
Date
Msg-id CA+Tgmoagx6qLCL1nTrNd6SHH8nvTHj8igvBWAwhk_rgtUfDoqQ@mail.gmail.com
Whole thread Raw
In response to Re: cache invalidation for PL/pgsql functions  (Marko Tiikkaja <marko@joh.to>)
Responses Re: cache invalidation for PL/pgsql functions  (Fabrízio de Royes Mello <fabriziomello@gmail.com>)
Re: cache invalidation for PL/pgsql functions  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Fri, May 1, 2015 at 9:09 AM, Marko Tiikkaja <marko@joh.to> wrote:
> On 2015-04-28 19:43, Robert Haas wrote:
>> I guess
>> the root of the problem is that PL/plgsql's cache invalidation logic
>> only considers the pg_proc row's TID and xmin when deciding whether to
>> recompile.  For base types that's probably OK, but for composite
>> types, not so much.
>>
>> Thoughts?
>
> We recently hit a similar case in our production environment.  What was
> annoying about it is that there didn't seem to be a way for the application
> to fix the issue by itself, short of reconnecting; even DISCARD ALL doesn't
> help.  If we can't fix the underlying issue, can we at least provide a way
> for apps to invalidate these caches themselves, for example in the form of a
> DISCARD option?

It's been discussed before and I am in favor of it.  However the
implementation is a bit challenging.  The DISCARD command doesn't know
what PLs may have decided to cache, nonwithstanding the fact that they
all cache basically the same stuff using basically the same method.  I
think the PL interface will need to be extended in some way to support
a new callback.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: procost for to_tsvector
Next
From: Robert Haas
Date:
Subject: Re: initdb -S and tablespaces