| From: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> | 
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> | 
| Cc: | Aleksander Alekseev <aleksander(at)timescale(dot)com>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org> | 
| Subject: | Re: rand48 replacement | 
| Date: | 2021-09-25 20:08:57 | 
| Message-ID: | alpine.DEB.2.22.394.2109251929180.3216312@pseudo | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers | 
Hello Tom,
> Just FTR, I strongly object to your removal of process-startup srandom()
> calls.
Ok. The point of the patch is to replace and unify the postgres underlying 
PRNG, so there was some logic behind this removal.
> Those are not only setting the seed for our own use, but also ensuring 
> that things like random() calls within PL functions or other libraries 
> aren't 100% predictable.
Sure, they shouldn't be predictable.
Attached v15 also does call srandom if it is there, and fixes yet another 
remaining random call.
-- 
Fabien.
| Attachment | Content-Type | Size | 
|---|---|---|
| prng-15.patch | text/x-diff | 68.4 KB | 
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tomas Vondra | 2021-09-25 21:01:21 | Re: extended stats on partitioned tables | 
| Previous Message | Hannu Krosing | 2021-09-25 20:05:43 | Re: logical decoding and replication of sequences |