COPY TO FORMAT JSON: respect column list order

When COPY TO with FORMAT json is given an explicit column list that
names all columns in a different order, the JSON output incorrectly
used the table's physical column order instead of the requested order.

This happened because BeginCopyTo() only built a restricted TupleDesc
when list_length(attnumlist) < tupDesc->natts.  When all columns are
listed (just reordered), this condition was false and no projected
TupleDesc was built, causing CopyToJsonOneRow() to emit columns in
physical order.

Fix by also building the projected TupleDesc when an explicit column
list was provided (attnamelist != NIL), even if it names all columns.

Author: Baji Shaik <[email protected]>
Reviewed-by: Andrew Dunstan <[email protected]>
Discussion: 
https://postgr.es/m/ca+fm-rod4cnkm524n6edgtz9xozohjdnv8j_9mvr2+2t1qw...@mail.gmail.com

Branch
------
master

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

Modified Files
--------------
src/backend/commands/copyto.c      | 14 +++++++++-----
src/test/regress/expected/copy.out |  9 +++++++++
src/test/regress/sql/copy.sql      |  9 +++++++++
3 files changed, 27 insertions(+), 5 deletions(-)

Reply via email to