Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

WIP: [core] add uuidv7 support #2746

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

WIP: [core] add uuidv7 support #2746

wants to merge 2 commits into from

Conversation

seven1240
Copy link
Collaborator

@seven1240 seven1240 commented Jan 20, 2025 •
edited
Loading

this is a different implementation than #2734

@seven1240
Copy link
Collaborator Author

v7 looks much faster than v4

2025-01-20 22:45:34.157636 100.00% [DEBUG] switch_core.c:599 test_create_uuid_speed:
2025-01-20 22:45:34.157636 100.00% [DEBUG] switch_core.c:610 10000 uuidv7_new running time: 0.000832 seconds
2025-01-20 22:45:34.196881 100.00% [DEBUG] switch_core.c:619 10000 uuid_generate running time: 0.036450 seconds
2025-01-20 22:45:34.196881 100.00% [DEBUG] switch_core.c:610 100000 uuidv7_new running time: 0.005779 seconds
2025-01-20 22:45:34.440331 99.90% [DEBUG] switch_core.c:619 100000 uuid_generate running time: 0.267279 seconds
2025-01-20 22:45:34.519784 99.90% [DEBUG] switch_core.c:610 1000000 uuidv7_new running time: 0.050283 seconds
2025-01-20 22:45:37.200556 98.83% [DEBUG] switch_core.c:619 1000000 uuid_generate running time: 2.718588 seconds

@seven1240 seven1240 mentioned this pull request Jan 20, 2025
Open
@seven1240 seven1240 requested a review from andywolk January 24, 2025 00:22
@ar45
Copy link
Contributor

ar45 commented Jan 24, 2025

Great patch

@seven1240 seven1240 changed the title [core] add uuidv7 support WIP: [core] add uuidv7 support Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andywolk andywolk Awaiting requested review from andywolk

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@seven1240 @ar45