Re: [PATCH] Add result_types column to pg_prepared_statements view

From: Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>
To: Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: [PATCH] Add result_types column to pg_prepared_statements view
Date: 2022-07-05 09:27:54
Message-ID: 87mtdnkixx.fsf@wibble.ilmari.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> writes:

> There was a problem that we didn't cover: Not all prepared statements
> have result descriptors (e.g., DML statements), so that would crash as
> written.

D'oh!

> I have changed it to return null for result_types in that case, and
> added a test case.

Thanks for spotting and fixing that.

- ilmari

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tatsuo Ishii 2022-07-05 10:37:02 Re: Wrong provolatile value for to_timestamp (1 argument)
Previous Message Peter Eisentraut 2022-07-05 09:20:53 Re: [PATCH] Add result_types column to pg_prepared_statements view