RE: v12 and pg_restore -f-
imai.yoshikazu@fujitsu.com <imai.yoshikazu@fujitsu.com>
From: "imai.yoshikazu@fujitsu.com" <imai.yoshikazu@fujitsu.com>
To: 'Stephen Frost' <sfrost@snowman.net>, Euler Taveira <euler@timbira.com.br>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Andrew Gierth <andrew@tao11.riddles.org.uk>, Justin Pryzby <pryzby@telsasoft.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2019-10-16T06:25:33Z
Lists: pgsql-hackers, pgsql-general
Hi, On Sun, Oct 6, 2019 at 7:09 PM, Justin Pryzby wrote: > I saw this and updated our scripts with pg_restore -f- > https://www.postgresql.org/docs/12/release-12.html > |In pg_restore, require specification of -f - to send the dump contents to standard output (Euler Taveira) > |Previously, this happened by default if no destination was specified, but that was deemed to be unfriendly. > > What I didn't realize at first is that -f- has no special meaning in v11 - it > just writes a file called ./- And it's considered untennable to change behavior of v11. Ahh... I totally missed thinking about the behavior of "-f -" in v11 when I reviewed this patch. On Wed, Oct 9, 2019 at 0:45 PM, Stephen Frost wrote: > * Euler Taveira (euler@timbira.com.br) wrote: > > Em ter, 8 de out de 2019 às 15:08, Stephen Frost <sfrost@snowman.net> escreveu: > > > * Tom Lane (tgl@sss.pgh.pa.us) wrote: > > > > Andrew Gierth <andrew@tao11.riddles.org.uk> writes: > > > > > "Tom" == Tom Lane <tgl@sss.pgh.pa.us> writes: > > > > > Tom> Perhaps we could change the back branches so that they > > > > > interpret Tom> "-f -" as "write to stdout", but without > > > > > enforcing that you use Tom> that syntax. > > > > > > > > > We should definitely do that. > > > > > > I agree that this would be a reasonable course of action. Really, > > > it should have always meant that... > > > > > Indeed, it was a broken behavior and the idea was to fix it. However, > > changing pg_restore in back-branches is worse than do nothing because > > it could break existent scripts. > > I can certainly respect that argument, in general, but in this specific case, I've got a really hard time believeing > that people wrote scripts which use '-f -' with the expectation that a './-' file was to be created. +1. If we only think of the problem that we can't use "-f -" with the meaning "dump to the stdout" in v11 and before ones, it seems a bug and we should fix it. Of course, if we fix it, some people would go into the trouble, but such people are who wrote scripts which use '-f -' with the expectation that a './-' file. I don't think there are such people a lot. -- Yoshikazu Imai
Commits
-
Change pg_restore -f- to dump to stdout instead of to ./-
- 5ee8f0fe13b4 10.11 landed
- d38635725cad 9.5.20 landed
- 12a51e2ebea7 9.6.16 landed
- 9fb25fda6d56 9.4.25 landed
- 3574c0ac0509 11.6 landed