pgsql: Refactor ATExecAddColumn() to use BuildDescForRelation()

Поиск
Список
Период
Сортировка
От Peter Eisentraut
Тема pgsql: Refactor ATExecAddColumn() to use BuildDescForRelation()
Дата
Msg-id E1rOJIc-0015ow-Cq@gemulon.postgresql.org
обсуждение исходный текст
Список pgsql-committers
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

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/a1604237a6ffee70b171bacd5f36b0e380afd33a

Modified Files
--------------
src/backend/commands/tablecmds.c | 89 ++++++++++------------------------------
1 file changed, 22 insertions(+), 67 deletions(-)


В списке pgsql-committers по дате отправления:

Предыдущее
От: Robert Haas
Дата:
Сообщение: pgsql: Fix typo.
Следующее
От: Jeff Davis
Дата:
Сообщение: pgsql: Re-validate connection string in libpqrcv_connect().