On Fri, Apr 7, 2017 at 7:35 AM, Tom Lane <[email protected]> wrote: > Amit Langote <[email protected]> writes: >> ... One of >> the earlier versions of that patch introduced a consider_typeid parameter >> for which only ExecEvalConvertRowtype() passed true. > > Yeah, I was thinking of adding a flag along that line to fix this, but > desisted after figuring out that ExecEvalConvertRowtype was the only > candidate for using it. Ashutosh's patch had already shown that it'd > be better to pass "false" there too, so we'd end up with no use cases > at all.
Probably we should also add an assertion there to make sure ExecEvalConvertRowtype never gets same input and output types. If that's the case, we don't need the copy as well. -- Best Wishes, Ashutosh Bapat EnterpriseDB Corporation The Postgres Database Company -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
