public inbox for [email protected]help / color / mirror / Atom feed
[PATCH 03/18] pg_restore: "must be specified" and --list 4+ messages / 3 participants [nested] [flat]
* [PATCH 03/18] pg_restore: "must be specified" and --list @ 2020-12-06 04:43 Justin Pryzby <[email protected]> 0 siblings, 0 replies; 4+ messages in thread From: Justin Pryzby @ 2020-12-06 04:43 UTC (permalink / raw) This was discussed here, but the idea got lost. https://www.postgresql.org/message-id/flat/20190612170201.GA11881%40alvherre.pgsql#2984347ab074e6f19... --- src/bin/pg_dump/pg_restore.c | 2 +- src/bin/pg_dump/t/001_basic.pl | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/bin/pg_dump/pg_restore.c b/src/bin/pg_dump/pg_restore.c index 589b4aed53..f6e6e41329 100644 --- a/src/bin/pg_dump/pg_restore.c +++ b/src/bin/pg_dump/pg_restore.c @@ -305,7 +305,7 @@ main(int argc, char **argv) /* Complain if neither -f nor -d was specified (except if dumping TOC) */ if (!opts->cparams.dbname && !opts->filename && !opts->tocSummary) { - pg_log_error("one of -d/--dbname and -f/--file must be specified"); + pg_log_error("one of -d/--dbname, -f/--file, or -l/--list must be specified"); exit_nicely(1); } diff --git a/src/bin/pg_dump/t/001_basic.pl b/src/bin/pg_dump/t/001_basic.pl index 083fb3ad08..8280914c2a 100644 --- a/src/bin/pg_dump/t/001_basic.pl +++ b/src/bin/pg_dump/t/001_basic.pl @@ -63,8 +63,8 @@ command_fails_like( command_fails_like( ['pg_restore'], - qr{\Qpg_restore: error: one of -d/--dbname and -f/--file must be specified\E}, - 'pg_restore: error: one of -d/--dbname and -f/--file must be specified'); + qr{\Qpg_restore: error: one of -d/--dbname, -f/--file, or -l/--list must be specified\E}, + 'pg_restore: error: one of -d/--dbname, -f/--file, or -l/--list must be specified'); command_fails_like( [ 'pg_restore', '-s', '-a', '-f -' ], -- 2.17.0 --lc9FT7cWel8HagAv Content-Type: text/x-diff; charset=us-ascii Content-Disposition: attachment; filename="0004-typos-in-master.patch" ^ permalink raw reply [nested|flat] 4+ messages in thread
* random() function documentation @ 2022-04-11 18:38 Dagfinn Ilmari Mannsåker <[email protected]> 2022-04-11 18:45 ` Re: random() function documentation Tom Lane <[email protected]> 0 siblings, 1 reply; 4+ messages in thread From: Dagfinn Ilmari Mannsåker @ 2022-04-11 18:38 UTC (permalink / raw) To: [email protected] Hi Hackers, I just noticed that the since the random() rewrite¹, the documentation's claim² that it "uses a simple linear congruential algorithm" is no longer accurate (xoroshiro128** is an xorshift variant, which is a linear-feedback shift register algorithm). I don't have a suggestion for the exact wording, since I don't know whether xoroshiro128** qualifies as "simple", or to what level of specificity we want to document the algorithm. - ilmari [1] https://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=3804539e48e794781c6145c7f988f5d507418... [2] https://www.postgresql.org/docs/devel/functions-math.html#FUNCTIONS-MATH-RANDOM-TABLE ^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: random() function documentation 2022-04-11 18:38 random() function documentation Dagfinn Ilmari Mannsåker <[email protected]> @ 2022-04-11 18:45 ` Tom Lane <[email protected]> 2022-04-11 19:00 ` Re: random() function documentation Dagfinn Ilmari Mannsåker <[email protected]> 0 siblings, 1 reply; 4+ messages in thread From: Tom Lane @ 2022-04-11 18:45 UTC (permalink / raw) To: Dagfinn Ilmari Mannsåker <[email protected]>; +Cc: [email protected] =?utf-8?Q?Dagfinn_Ilmari_Manns=C3=A5ker?= <[email protected]> writes: > I just noticed that the since the random() rewrite¹, the documentation's > claim² that it "uses a simple linear congruential algorithm" is no > longer accurate (xoroshiro128** is an xorshift variant, which is a > linear-feedback shift register algorithm). > I don't have a suggestion for the exact wording, since I don't know > whether xoroshiro128** qualifies as "simple", or to what level of > specificity we want to document the algorithm. How about we just say "uses a linear-feedback shift register algorithm"? "Simple" is in the eye of the beholder anyway. regards, tom lane ^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: random() function documentation 2022-04-11 18:38 random() function documentation Dagfinn Ilmari Mannsåker <[email protected]> 2022-04-11 18:45 ` Re: random() function documentation Tom Lane <[email protected]> @ 2022-04-11 19:00 ` Dagfinn Ilmari Mannsåker <[email protected]> 0 siblings, 0 replies; 4+ messages in thread From: Dagfinn Ilmari Mannsåker @ 2022-04-11 19:00 UTC (permalink / raw) To: Tom Lane <[email protected]>; +Cc: [email protected] Tom Lane <[email protected]> writes: > =?utf-8?Q?Dagfinn_Ilmari_Manns=C3=A5ker?= <[email protected]> writes: >> I just noticed that the since the random() rewrite¹, the documentation's >> claim² that it "uses a simple linear congruential algorithm" is no >> longer accurate (xoroshiro128** is an xorshift variant, which is a >> linear-feedback shift register algorithm). > >> I don't have a suggestion for the exact wording, since I don't know >> whether xoroshiro128** qualifies as "simple", or to what level of >> specificity we want to document the algorithm. > > How about we just say "uses a linear-feedback shift register algorithm"? That works for me. Nice and simple, and not overly specific. Should we perhaps also add a warning that the same seed is not guaranteed to produce the same sequence across different (major?) versions? > "Simple" is in the eye of the beholder anyway. Indeed. > regards, tom lane - ilmari ^ permalink raw reply [nested|flat] 4+ messages in thread
end of thread, other threads:[~2022-04-11 19:00 UTC | newest] Thread overview: 4+ messages (download: mbox mbox.gz follow: Atom feed) -- links below jump to the message on this page -- 2020-12-06 04:43 [PATCH 03/18] pg_restore: "must be specified" and --list Justin Pryzby <[email protected]> 2022-04-11 18:38 random() function documentation Dagfinn Ilmari Mannsåker <[email protected]> 2022-04-11 18:45 ` Re: random() function documentation Tom Lane <[email protected]> 2022-04-11 19:00 ` Re: random() function documentation Dagfinn Ilmari Mannsåker <[email protected]>
This inbox is served by agora; see mirroring instructions for how to clone and mirror all data and code used for this inbox