On Fri, Aug 15, 2025 at 11:44 AM Masahiko Sawada <sawada.m...@gmail.com> wrote: > > On Thu, Aug 14, 2025 at 5:10 AM Erik Nordström <e...@tigerdata.com> wrote: > > > > > > > > On Wed, Aug 13, 2025 at 11:52 AM Andrey Borodin <x4...@yandex-team.ru> > > wrote: > >> > >> Hi Erik! > >> > >> > On 13 Aug 2025, at 11:05, Erik Nordström <e...@tigerdata.com> wrote: > >> > > >> > I think I found a small typo in the function that extracts a timestamp > >> > from a UUIDv7 (uuid_extract_timestamp). Unless I am mistaken, the > >> > constant US_PER_MS should be used instead of NS_PER_US when converting > >> > milliseconds to microseconds. Fortunately, these constants are the same > >> > so the calculation is still correct. > >> > >> Wow, that's a very good level of proofreading! Yes, you are correct, it's > >> must be US_PER_MS. > >> > >> > > >> > Anyway, attaching a patch to fix this typo. > >> > >> LGTM. > > > > > > Ok, maybe a committer would like to commit this change immediately? > > Thank you for the report! I'll push the patch to master and v18 shortly.
Pushed. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
