Re: pgsql: Set random seed for pgbench. - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Set random seed for pgbench.
Date
Msg-id 32390.1522464534@sss.pgh.pa.us
Whole thread Raw
In response to pgsql: Set random seed for pgbench.  (Teodor Sigaev <teodor@sigaev.ru>)
Responses Re: pgsql: Set random seed for pgbench.  (Fabien COELHO <coelho@cri.ensmp.fr>)
List pgsql-committers
Teodor Sigaev <teodor@sigaev.ru> writes:
> Set random seed for pgbench.

So this patch has ignored the possibility of not having pg_strong_random.
That's moving the portability goalposts in a way that I do not find
acceptable for such a marginal feature.  What behavior do you think we
should have for platforms without that --- accept the seed=rand option
but then error out, or not recognize the option at all?

BTW, I also note that the patch is constructing error messages from
sentence fragments, which is verboten per translatability policy.
While that's perhaps not too important as long as we don't have message
translations for pgbench, it still seems like something to fix now not
later.

            regards, tom lane


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Portability fix for commit 9a895462d.
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Transforms for jsonb to PL/Python