pgsql: Fix catalog data of pg_stop_backup(), labelled v2

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Fix catalog data of pg_stop_backup(), labelled v2
Date: 2022-03-03 01:53:00
Message-ID: E1nPaeK-000f8H-AQ@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix catalog data of pg_stop_backup(), labelled v2

This function has been incorrectly marked as a set-returning function
with prorows (estimated number of rows) set to 1 since its creation in
7117685, that introduced non-exclusive backups. There is no need for
that as the function is designed to return only one tuple.

This commit fixes the catalog definition of pg_stop_backup_v2() so as it
is not marked as proretset anymore, with prorows set to 0. This
simplifies its internals by removing one tuplestore (used for one single
record anyway) and by removing all the checks related to a set-returning
function.

Issue found during my quest to simplify some of the logic used in
in-core system functions.

Bump catalog version.

Reviewed-by: Aleksander Alekseev, Kyotaro Horiguchi
Discussion: https://postgr.es/m/Yh8guT78f1Ercfzw@paquier.xyz

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/62ce0c758d5d66092efbca7d037233e2ca9bdc78

Modified Files
--------------
src/backend/access/transam/xlogfuncs.c | 36 ++++++--------------------------
src/backend/catalog/system_functions.sql | 2 +-
src/include/catalog/catversion.h | 2 +-
src/include/catalog/pg_proc.dat | 6 +++---
4 files changed, 11 insertions(+), 35 deletions(-)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2022-03-03 15:09:40 Re: pgsql: Fix catalog data of pg_stop_backup(), labelled v2
Previous Message Tom Lane 2022-03-02 16:57:21 pgsql: Allow root-owned SSL private keys in libpq, not only the backend