On Mon, Dec 9, 2024 at 2:23 PM Andres Freund <and...@anarazel.de> wrote: > > Hi, > > On 2024-12-09 14:10:30 -0800, Masahiko Sawada wrote: > > While reviewing UUIDv7 patch[1], I realized gen_random_uuid() is > > marked leakproof even though it doesn't take arguments. The functions > > without arguments don't need to be marked leakproof in principle. This > > is the sole function that has no arguments and is listed in the "List > > of built-in leakproof functions" in opr_sanity.sql. I've attached the > > patch for fixing it and for better consistency with new UUID > > generation functions discussed on that thread. > > Seems like it'd make sense to add a test to opr_sanity.sql so we don't > reintroduce such cases? >
Thank you for the comment. It's a good idea. I've updated the patch. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
v2-0001-Unmark-gen_random_uuid-function-leakproof.patch
Description: Binary data