Refactor ATExecAddColumn() to use BuildDescForRelation()
authorPeter Eisentraut <peter@eisentraut.org>
Fri, 12 Jan 2024 15:05:15 +0000 (16:05 +0100)
committerPeter Eisentraut <peter@eisentraut.org>
Fri, 12 Jan 2024 15:11:29 +0000 (16:11 +0100)
commita1604237a6ffee70b171bacd5f36b0e380afd33a
tree12a32e566f9cb87397f4a86dce47ede1f0a3a5bf
parent882887e9b458f533b301c36951c93dfdd37dccfc
Refactor ATExecAddColumn() to use BuildDescForRelation()

BuildDescForRelation() has all the knowledge for converting a
ColumnDef into pg_attribute/tuple descriptor.  ATExecAddColumn() can
make use of that, instead of duplicating all that logic.  We just pass
a one-element list of ColumnDef and we get back exactly the data
structure we need.  Note that we don't even need to touch
BuildDescForRelation() to make this work.

Reviewed-by: Alvaro Herrera <alvherre@alvh.no-ip.org>
Discussion: https://www.postgresql.org/message-id/flat/52a125e4-ff9a-95f5-9f61-b87cf447e4da@eisentraut.org
src/backend/commands/tablecmds.c