Re: uuidv7 improperly accepts dates before 1970-01-01

On Thu, Jul 16, 2026 at 8:01 AM Baji Shaik <baji(dot)pgdev(at)gmail(dot)com> wrote:
>
> On Tue, Jul 14, 2026 at 2:16 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
>>
>>
>> I guess we don't need to name it with the prefix UUIDV7 as the offset
>> value is actually not limited to UUIDv7. How about renaming it to
>> PG_UNIX_EPOCH_OFFSET_US?
>>
>> +-- uuidv7: a large but in-range forward shift is accepted
>> +SELECT uuid_extract_timestamp(uuidv7('1000 years'::interval)) > now()
>> + '999 years'::interval;
>>
>> I've merged two patches into one and made some cosmetic changes and
>> the above suggestions. I'm going to push it, barring objections.
>
>
> Thanks for the review and the cleanup! v6 looks good to me.
>
> The rename to PG_UNIX_EPOCH_OFFSET_US makes sense, and replacing
> the 292230 years test with the in-range forward shift is a nice improvement.
>
> No objections from my side.

Thank you for reviewing the patch. Pushed.

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com