From e434e21e114b423e919324ad6ce1f3f079ca2a03 Mon Sep 17 00:00:00 2001 From: Michael Paquier Date: Sat, 9 Sep 2023 21:12:41 +0900 Subject: [PATCH] Remove redundant assignments in copyfrom.c The tuple descriptor and the number of attributes are assigned twice to the same values in BeginCopyFrom(), for what looks like a small thinko coming from the refactoring done in c532d15dddff1. Author: Jingtang Zhang Discussion: https://postgr.es/m/CAPsk3_CrYeXUVHEiaWAYxY9BKiGvGT3AoXo_+Jm0xP_s_VmXCA@mail.gmail.com --- src/backend/commands/copyfrom.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/backend/commands/copyfrom.c b/src/backend/commands/copyfrom.c index b47cb5c66d..70871ed819 100644 --- a/src/backend/commands/copyfrom.c +++ b/src/backend/commands/copyfrom.c @@ -1533,8 +1533,6 @@ BeginCopyFrom(ParseState *pstate, cstate->rteperminfos = pstate->p_rteperminfos; } - tupDesc = RelationGetDescr(cstate->rel); - num_phys_attrs = tupDesc->natts; num_defaults = 0; volatile_defexprs = false;