From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
---|---|
To: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: tablecmds.c/MergeAttributes() cleanup |
Date: | 2023-08-29 12:09:31 |
Message-ID: | 20230829120931.gudur2e4kupeihg3@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2023-Aug-29, Peter Eisentraut wrote:
> From 471fda80c41fae835ecbe63ae8505526a37487a9 Mon Sep 17 00:00:00 2001
> From: Peter Eisentraut <peter(at)eisentraut(dot)org>
> Date: Wed, 12 Jul 2023 16:12:35 +0200
> Subject: [PATCH v2 04/10] Add TupleDescGetDefault()
>
> This unifies some repetitive code.
>
> Note: I didn't push the "not found" error message into the new
> function, even though all existing callers would be able to make use
> of it. Using the existing error handling as-is would probably require
> exposing the Relation type via tupdesc.h, which doesn't seem
> desirable.
Note that all errors here are elog(ERROR), not user-facing, so ISTM
it would be OK to change them to report the relation OID rather than the
name.
--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
"In fact, the basic problem with Perl 5's subroutines is that they're not
crufty enough, so the cruft leaks out into user-defined code instead, by
the Conservation of Cruft Principle." (Larry Wall, Apocalypse 6)
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Bapat | 2023-08-29 12:10:25 | Re: persist logical slots to disk during shutdown checkpoint |
Previous Message | Hayato Kuroda (Fujitsu) | 2023-08-29 11:58:31 | RE: [PoC] pg_upgrade: allow to upgrade publisher node |