It would be useful to add a standard comparative benchmark with several
parameters and use cases to the patch, so that IT departments can compare
UUIDv7, ULID, UUIDv4, Snowflake ID and BIGSERIAL for their hardware and
conditions.
I know for a fact that IT departments make such benchmarks of low quality. They
usually measure the generation rate, which is meaningless because it is usually
excessive. It makes sense to measure the rate of single-threaded and
multi-threaded insertion of a large number of records (with and without
partitioning), as well as the rate of execution of queries to join big tables,
to update or delete a large number of records. It is important to measure
memory usage, processor load, etc.
Sergey Prokhorenko sergeyprokhore...@yahoo.com.au
On Wednesday 27 November 2024 at 09:24:40 pm GMT+3, Masahiko Sawada
<sawada.m...@gmail.com> wrote:
On Tue, Nov 26, 2024 at 7:11 PM Andrey Borodin <x4...@yandex-team.ru> wrote:
>
>
>
> > On 27 Nov 2024, at 04:11, Masahiko Sawada <sawada.m...@gmail.com> wrote:
> >
> > On Tue, Nov 26, 2024 at 1:55 PM Jelte Fennema-Nio <postg...@jeltef.nl>
> > wrote:
> >>
> >> On Tue, 26 Nov 2024 at 21:48, Sergey Prokhorenko
> >> <sergeyprokhore...@yahoo.com.au> wrote:
> >>> gen_uuidv7() is OK
> >>
> >> I'd very much prefer to not have a gen_ or get_ prefix as argued
> >> before[1][2].
> >>
> >> My vote is still for simply uuidv7() and uuidv4()
> >>
> >>> uuid-ossp is outdated, slow and not supported by the author. UUIDv7 is
> >>> the renaissance of UUIDs. So we should not depend on legacy technology
> >>> names
> >>
> >> agreed
> >>
> >
> > It seems that we agreed to use 'uuidv7' instead of 'uuid_v7()'. There
> > is discussion whether we should add 'gen_' or 'get_' but let's go back
> > to the previously-agreed function name 'uuidv7()' for now. We can
> > rename it later if we find a better name.
>
> I think uuidv7() is kind of consensual.
>
> > I've attached the new version patch that incorporated all comments and
> > renamed the functions. Also I avoided using 'if defined(__darwin__) ||
> > defined(_MSC_VER)' twice.
>
> Good, I think now it's a bit easier to understand those 2 bits.
>
Thanks.
I'm going to push the v39 patch (after self review again), barring any
objections and further comments.
Regards,
--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com
