Re: Remove unused argument from ApplyLogicalMappingFile()

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com>
Cc: Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Remove unused argument from ApplyLogicalMappingFile()
Date: 2026-01-30 00:07:04
Message-ID: CAHGQGwF6jLaYG6ZoyAovqMkzt0_yWSh2s69_RQ4mUkJjQebVzw@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jan 28, 2026 at 6:23 PM Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
>
> On Wed, Jan 28, 2026 at 2:18 PM Hayato Kuroda (Fujitsu)
> <kuroda(dot)hayato(at)fujitsu(dot)com> wrote:
> >
> > Dear Nagata-san,
> >
> > > I noticed that ApplyLogicalMappingFile() has an unused argument.
> > > If there is no specific reason for keeping it, I think it could be
> > > removed. I have attached a small patch that removes this argument.
>
> Thanks for the patch! I will commit it.

I've pushed the patch. Thanks!

Regards,

--
Fujii Masao

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Mark Wong 2026-01-30 00:15:18 Re: updates for handling optional argument in system functions
Previous Message Alexander Borisov 2026-01-30 00:04:33 Re: Non-committer reviews: is it helpful?