diff options
| author | Peter Eisentraut | 2023-08-31 09:11:59 +0000 |
|---|---|---|
| committer | Peter Eisentraut | 2023-08-31 09:11:59 +0000 |
| commit | d7ceb41b9b84092517bbd7aa1ea6c446f023504e (patch) | |
| tree | bebed7b5389106da9211065f7eef59fd60f0047c /src/backend | |
| parent | f94dec76cc7aa1dc1636708e81581f9b84e7e15c (diff) | |
Correct ObjectProperty entry for transforms
There was some confusion in the ObjectProperty entry for transforms.
Some fields had values that were apparently meant for a different
field. Also, some fields were not assigned, which is okay for most
fields, but not for all. In particular, for .oid_catcache_id,
.name_catcache_id, and .objtype, zero is a valid value, so we need to
use -1 if not applicable. It has apparently been like that from the
very beginning (commit cac7658205). The faulty values were not
actually reachable, so it's not a big problem in practice, but we
should make it correct.
Discussion: https://www.postgresql.org/message-id/flat/75ae5875-3abc-dafc-8aec-73247ed41cde@eisentraut.org
Diffstat (limited to 'src/backend')
| -rw-r--r-- | src/backend/catalog/objectaddress.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/backend/catalog/objectaddress.c b/src/backend/catalog/objectaddress.c index 95fefc7565b..715201f5a25 100644 --- a/src/backend/catalog/objectaddress.c +++ b/src/backend/catalog/objectaddress.c @@ -462,8 +462,14 @@ static const ObjectPropertyType ObjectProperty[] = TransformRelationId, TransformOidIndexId, TRFOID, + -1, + Anum_pg_transform_oid, + InvalidAttrNumber, + InvalidAttrNumber, InvalidAttrNumber, - Anum_pg_transform_oid + InvalidAttrNumber, + -1, + false }, { "trigger", |