Re: Discarding DISCARD ALL

From: Andreas Karlsson <andreas(at)proxel(dot)se>
To: Simon Riggs <simon(at)2ndquadrant(dot)com>
Cc: Vladimir Sitnikov <sitnikov(dot)vladimir(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Discarding DISCARD ALL
Date: 2020-12-23 22:34:32
Message-ID: 125b1e8a-8b7f-b4b1-19ac-aea0ce15c0a1@proxel.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 12/23/20 6:49 PM, Simon Riggs wrote:
> The whole premise of the patch is tighter integration, with the server
> providing the facilities that poolers need.

I am all for that. Ideally I would want builtin connection pooling but
short term I think the way forward is most likely tighter integration.

> The patch can be enhanced to do whatever else we agree is desirable.
>
> Do we need something like DISCARD ALL EXCEPT PREPARED STATEMENTS; ??
>
> If there are different requirements for each pooler, what are they?

If someone adds prepared statement support to e.g. PgBouncer that might
be a nice feature to have. Plus maybe something like "SET
transaction_cleanup = 'except_prepared'". But right now I think the
pools which support prepared statements do not use DISCARD ALL and
instead just trust the end user to not run SET or use temporary tables
which outlive transactions.

Andreas

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Justin Pryzby 2020-12-23 22:47:49 Re: Allow CLUSTER, VACUUM FULL and REINDEX to change tablespace on the fly
Previous Message Alvaro Herrera 2020-12-23 22:22:05 Re: Allow CLUSTER, VACUUM FULL and REINDEX to change tablespace on the fly