From: Tom Lane <tgl@sss.pgh.pa.us>
To: Stephen Frost <sfrost@snowman.net>
Cc: imai.yoshikazu@fujitsu.com <imai.yoshikazu@fujitsu.com>
Cc: Euler Taveira <euler@timbira.com.br>
Cc: Andrew Gierth <andrew@tao11.riddles.org.uk>
Cc: Justin Pryzby <pryzby@telsasoft.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: v12 and pg_restore -f-
Date: Thu, 17 Oct 2019 12:24:10 +0200
Message-ID: <19891.1571307850@sss.pgh.pa.us> (raw)
In-Reply-To: <20191016172148.GH6962@tamriel.snowman.net>
References: <20191006190839.GE18030@telsasoft.com>
<28770.1570394593@sss.pgh.pa.us>
<875zl11tla.fsf@news-spur.riddles.org.uk>
<30069.1570396524@sss.pgh.pa.us>
<20191008180840.GQ6962@tamriel.snowman.net>
<CAHE3wgi=VU0G9=S60=gQH+yyk8-Ke99dkt8Dp4OpZXkSu=67fQ@mail.gmail.com>
<20191009124505.GT6962@tamriel.snowman.net>
<OSBPR01MB4616298336E32F8EAEDA0FF394920@OSBPR01MB4616.jpnprd01.prod.outlook.com>
<20191016172148.GH6962@tamriel.snowman.net>
Stephen Frost <sfrost@snowman.net> writes:
> First, I'd like to clarify what I believe Tom's suggestion is, and then
> talk through that, as his vote sways this topic pretty heavily.
> Tom, I take it your suggestion is to have '-f -' be accepted to mean
> 'goes to stdout' in all branches?
Yes.
> That goes against the argument that
> we don't want to break existing scripts, as it's possible that there are
> existing scripts that depend on '-f -' actually going to a './-' file.
While that's theoretically possible, I think that the number of cases
where somebody is actually expecting that is epsilon. It seems more
useful to tell people that they can now use "-f -" in all branches,
and it's required to use it as of v12.
Alternatively, we could revoke the requirement to use "-f -" in 12,
and wait a couple releases before enforcing it. The fundamental
problem here is that we tried to go from "-f - doesn't work" to
"you must use -f -" with no grace period where "-f - is optional".
In hindsight that was a bad idea.
> If you meant for all branches to accept '-f -' and have it go to a './-'
> file then that's just a revert of this entire change, which I can't
> agree with either
No, I'm not proposing a full revert. But there's certainly room to
consider reverting the part that says you *must* write "-f -" to get
output to stdout.
regards, tom lane
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-general@postgresql.org
Cc: tgl@sss.pgh.pa.us, sfrost@snowman.net, imai.yoshikazu@fujitsu.com, euler@timbira.com.br, andrew@tao11.riddles.org.uk, pryzby@telsasoft.com, pgsql-hackers@lists.postgresql.org
Subject: Re: v12 and pg_restore -f-
In-Reply-To: <19891.1571307850@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