Horton

Re: remove unneeded pstrdup in fetch_table_list

Daniel Gustafsson <daniel@yesql.se>

View thread View on postgr.es
From: Daniel Gustafsson <daniel@yesql.se>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: "Hou, Zhijie" <houzj.fnst@cn.fujitsu.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-01-13T13:23:26Z
Lists: pgsql-hackers 
> On 13 Jan 2021, at 14:09, Amit Kapila <amit.kapila16@gmail.com> wrote:
> 
> On Wed, Jan 13, 2021 at 2:55 PM Daniel Gustafsson <daniel@yesql.se> wrote:
>> 
>>> On 13 Jan 2021, at 07:10, Amit Kapila <amit.kapila16@gmail.com> wrote:
>> 
>>> Your observation seems correct to me, though I have not tried to test
>>> your patch.
>> 
>> +1 to apply, this looks correct and passes tests.  Scanning around I didn't see
>> any other occurrences of the same pattern.
> 
> Thanks. I am thinking to backpatch this even though there is no
> problem reported from any production system. What do you think?

No objections from me.

cheers ./daniel

Commits

  1. Remove unnecessary pstrdup in fetch_table_list.