pgsql: Move BuildDescForRelation() from tupdesc.c to tablecmds.c - Mailing list pgsql-committers

From Peter Eisentraut
Subject pgsql: Move BuildDescForRelation() from tupdesc.c to tablecmds.c
Date
Msg-id E1qoPRf-000BDE-6z@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Move BuildDescForRelation() from tupdesc.c to tablecmds.c

BuildDescForRelation() main job is to convert ColumnDef lists to
pg_attribute/tuple descriptor arrays, which is really mostly an
internal subroutine of DefineRelation() and some related functions,
which is more the remit of tablecmds.c and doesn't have much to do
with the basic tuple descriptor interfaces in tupdesc.c.  This is also
supported by observing the header includes we can remove in tupdesc.c.
By moving it over, we can also (in the future) make
BuildDescForRelation() use more internals of tablecmds.c that are not
sensible to be exposed in tupdesc.c.

Discussion: https://www.postgresql.org/message-id/flat/52a125e4-ff9a-95f5-9f61-b87cf447e4da@eisentraut.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/04e485273b56bec0e4637f97c587eca26cf823f0

Modified Files
--------------
src/backend/access/common/tupdesc.c | 109 +-----------------------------------
src/backend/commands/tablecmds.c    | 102 +++++++++++++++++++++++++++++++++
src/include/access/tupdesc.h        |   2 -
src/include/commands/tablecmds.h    |   2 +
4 files changed, 105 insertions(+), 110 deletions(-)


pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgsql: Refactor ListenSocket array.
Next
From: Michael Paquier
Date:
Subject: pgsql: worker_spi: Add tests for BGWORKER_BYPASS_ALLOWCONN