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.

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

Add UUIDv6 and UUIDv7 draft versions #29824

Closed
wants to merge 4 commits into from
Closed

Conversation

oittaa
Copy link

@oittaa oittaa commented Nov 28, 2021

https://github.com/uuid6/uuid6-ietf-draft/

A simple test case to show that these are actually sortable variants:

uuid6_1 = uuid6()
uuid7_1 = uuid7()
for _ in range(100000):
    uuid6_2 = uuid6()
    uuid7_2 = uuid7()
    assert uuid6_1 < uuid6_2
    assert uuid7_1 < uuid7_2
    uuid6_1, uuid7_1 = uuid6_2, uuid7_2

print(uuid6_1)
print(uuid7_1)

https://github.com/uuid6/uuid6-ietf-draft/

A simple test case to show that these are actually sortable variants:
```
uuid6_1 = uuid6()
uuid7_1 = uuid7()
for _ in range(100000):
    uuid6_2 = uuid6()
    uuid7_2 = uuid7()
    assert uuid6_1 < uuid6_2
    assert uuid7_1 < uuid7_2
    uuid6_1, uuid7_1 = uuid6_2, uuid7_2

print(uuid6_1)
print(uuid7_1)
```
@the-knights-who-say-ni

This comment was marked as resolved.

@github-actions
Copy link

This PR is stale because it has been open for 30 days with no activity.

@terryjreedy
Copy link
Member

Discussion on the issue and linked thereon suggests that a) the draft standard has changed, making this PR outdated, and b) that approval of the newer draft seems to be getting closer. @oittaa , do you have any intention of updating this?

@encukou
Copy link
Member

encukou commented Feb 28, 2024

I'll close this for now. The standard is still a draft; the PR has conflicts and is based on an older version.
If anyone wants to work on this, please open an issue first:

  • to separate the idea from the proposed implementation, and
  • so that any fix-up PRs can be linked to the issue.

Once the new UUIDs become an official standard, I don't think the feature will need wider discussion than an issue.

@encukou encukou closed this Feb 28, 2024
@grillazz
Copy link

@encukou loooks it is now "proposed standard" > https://www.rfc-editor.org/rfc/rfc9562

@hugovk
Copy link
Member

hugovk commented Jun 14, 2024

Which means we can now implement it:

https://en.wikipedia.org/wiki/Internet_Standard#Proposed_Standard

Let's continue in #102461 #89083.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review DO-NOT-MERGE stale Stale PR or inactive for long period of time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants