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

2021-12-26 Thread Fujii Masao
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

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

2021-12-24 Thread Kyotaro Horiguchi
At Fri, 24 Dec 2021 18:44:06 +0900 (JST), Kyotaro Horiguchi wrote in > At Fri, 24 Dec 2021 18:24:39 +0900, Fujii Masao > wrote in > > > > > > On 2021/12/24 18:00, Kyotaro Horiguchi wrote: > > > I saw the test has been revertd. > > > > Yes, I reverted the added unstable tests not to prevent

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

2021-12-24 Thread Kyotaro Horiguchi
At Fri, 24 Dec 2021 18:24:39 +0900, Fujii Masao wrote in > > > On 2021/12/24 18:00, Kyotaro Horiguchi wrote: > > I saw the test has been revertd. > > Yes, I reverted the added unstable tests not to prevent > buildfarm from testing other patches while I'm doing > the investigation. > > > http

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

2021-12-24 Thread Fujii Masao
On 2021/12/24 18:00, Kyotaro Horiguchi wrote: I saw the test has been revertd. Yes, I reverted the added unstable tests not to prevent buildfarm from testing other patches while I'm doing the investigation. https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=crake&dt=2021-12-24%2008%3A

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

2021-12-24 Thread Kyotaro Horiguchi
I saw the test has been revertd. At Fri, 24 Dec 2021 17:20:18 +0900, Fujii Masao wrote in > Hmm... some buildfarm animals reported a failure. So I will > investigate the issue. https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=crake&dt=2021-12-24%2008%3A02%3A25 > NOTICE: identifier > "

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

2021-12-24 Thread Fujii Masao
On 2021/12/24 16:58, Fujii Masao wrote: postgres_fdw: Allow postgres_fdw.application_name to include escape sequences. Hmm... some buildfarm animals reported a failure. So I will investigate the issue. Regards, -- Fujii Masao Advanced Computing Technology Center Research and Development H