pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Fujii Masao <masao.fujii@oss.nttdata.com>
Cc: kuroda.hayato@fujitsu.com <kuroda.hayato@fujitsu.com>
Cc: 'Masahiro Ikeda' <ikedamsh@oss.nttdata.com>
Cc: 'pgsql-hackers@lists.postgresql.org' <pgsql-hackers@lists.postgresql.org>
Subject: Re: Allow escape in application_name (was: [postgres_fdw] add local pid to fallback_application_name)
Date: Tue, 07 Sep 2021 18:46:06 -0400
Message-ID: <3220909.1631054766@sss.pgh.pa.us> (raw)
In-Reply-To: <6da5713e-9adf-5301-0d9d-eb6cce16ea40@oss.nttdata.com>
References: <TYCPR01MB5870D1E8B949DAF6D3B84E02F5F29@TYCPR01MB5870.jpnprd01.prod.outlook.com>
	<ed1c8ef2-16ac-4e56-d302-b1ff430baeec@oss.nttdata.com>
	<TYAPR01MB58661635D3884E3B863ED01CF5C79@TYAPR01MB5866.jpnprd01.prod.outlook.com>
	<TYAPR01MB5866EF9624D2DFA8A74E5E94F5CC9@TYAPR01MB5866.jpnprd01.prod.outlook.com>
	<8f985ba0-2e79-ab67-4fa4-d35024188047@oss.nttdata.com>
	<TYAPR01MB5866B697888D5568E6D1546BF5CD9@TYAPR01MB5866.jpnprd01.prod.outlook.com>
	<0d5df322-c9ad-308e-982b-a3d3275bcdc0@oss.nttdata.com>
	<TYAPR01MB5866A3CAE041F7B05E2EE3A9F5CE9@TYAPR01MB5866.jpnprd01.prod.outlook.com>
	<2dde164d-bc49-b3ca-0cc1-1c5fe40f0886@oss.nttdata.com>
	<TYAPR01MB58669D67A505AD8995BBD2D6F5CF9@TYAPR01MB5866.jpnprd01.prod.outlook.com>
	<21aa7786-fce9-5dbf-6b75-519015319724@oss.nttdata.com>
	<TYAPR01MB58666157D4FF274822A76B3BF5D29@TYAPR01MB5866.jpnprd01.prod.outlook.com>
	<42ea526c-aa4f-bfe2-5547-6fcf8ac6469c@oss.nttdata.com>
	<TYAPR01MB5866772D4732E9CD7C040D62F5D39@TYAPR01MB5866.jpnprd01.prod.outlook.com>
	<6da5713e-9adf-5301-0d9d-eb6cce16ea40@oss.nttdata.com>

Fujii Masao <masao.fujii@oss.nttdata.com> writes:
> Pushed 0001 patch. Thanks!

The buildfarm shows that this test case isn't terribly reliable.

TBH, I think you should just remove the test case altogether.
It does not look useful enough to justify a permanent expenditure of
testing cycles, never mind the amount of effort that would be needed to
stabilize it.

			regards, tom lane





view thread (18+ messages)  latest in thread

Message-ID: <3220909.1631054766@sss.pgh.pa.us>
Permalink:  ../3220909.1631054766@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/3220909.1631054766@sss.pgh.pa.us

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: tgl@sss.pgh.pa.us, masao.fujii@oss.nttdata.com, kuroda.hayato@fujitsu.com, ikedamsh@oss.nttdata.com, pgsql-hackers@lists.postgresql.org
  Subject: Re: Allow escape in application_name (was: [postgres_fdw] add local pid to fallback_application_name)
  In-Reply-To: <3220909.1631054766@sss.pgh.pa.us>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox