Re: pgsql: postgres_fdw: Allow postgres_fdw.application_name to include esc

From: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
To: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>
Cc: fujii(at)postgresql(dot)org, pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: postgres_fdw: Allow postgres_fdw.application_name to include esc
Date: 2021-12-27 04:44:05
Message-ID: 2ccd9631-60ea-c220-5a2a-0e064bd406af@oss.nttdata.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On 2021/12/24 18:44, Kyotaro Horiguchi wrote:
> I once thought the same, but that change causes buildfarms *always*
> truncate the resulting application name. I believe the base
> application_name and database_name never grows further longer. So I'd
> like to split the check to %a%u%% and %d%p.

Yeah, probably it's better to split the test in that way. So I attached the patch doing that at the original thread [1] in pgsql-hackers. Let's discuss this there.

[1]
https://postgr.es/m/e7b61420-a97b-8246-77c4-a0d48fba5a45@oss.nttdata.com

Regards,

--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2021-12-27 19:39:21 pgsql: Rename EmitWarningsOnPlaceholders() to MarkGUCPrefixReserved().
Previous Message Michael Paquier 2021-12-26 08:42:24 pgsql: Fix incorrect field count in pg_control_checkpoint()