pgsql: Remove now-dead code in StoreAttrDefault(). - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Remove now-dead code in StoreAttrDefault().
Date
Msg-id E1tpAdy-000cdu-0o@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Remove now-dead code in StoreAttrDefault().

StoreAttrDefault() is no longer responsible for filling
attmissingval, so remove the code for that.

Get rid of RawColumnDefault.missingMode, too, as we no longer
need that to pass information around.

While here, clean up some sloppy coding in StoreAttrDefault(),
such as failure to use XXXGetDatum macros.  These aren't bugs
but they're not good code either.

Reported-by: jian he <jian.universality@gmail.com>
Author: jian he <jian.universality@gmail.com>
Author: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/CACJufxHFssPvkP1we7WMhPD_1kwgbG52o=kQgL+TnVoX5LOyCQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/4528768d98f83070d625207dcb526982fb18f1ce

Modified Files
--------------
src/backend/catalog/heap.c       |  5 ++-
src/backend/catalog/pg_attrdef.c | 77 ++++------------------------------------
src/backend/commands/tablecmds.c |  8 ++---
src/include/catalog/heap.h       |  1 -
src/include/catalog/pg_attrdef.h |  3 +-
5 files changed, 11 insertions(+), 83 deletions(-)


pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: Re: pgsql: Set amcancrosscompare to true for hash
Next
From: Melanie Plageman
Date:
Subject: pgsql: Trigger more frequent autovacuums with relallfrozen