-
Notifications
You must be signed in to change notification settings - Fork 192
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
v7 should have a counter option? #717
Comments
Hi @rogusdev 👋 I'd be keen to go catch up on some of the discussion around that. Those sections look like recommendations rather than requirements, so I think It seems a bit overkill to me to need a clock, a source of randomness, and a monotonic counter to generate V7 UUIDs, but we could consider adding |
The test vector for UUIDv7 appears to use a fully random value for I think we should consider supporting this, but don't think we need to block stabilizing the current APIs on it. |
From my perspective, while the main point of uuid v7 is to put the clock at the front, so that db indexes can btree on them properly, counters add additional safety mechanisms that are quite to my taste: if you are generating ids in a single thread, you are guaranteed that no duplicates can happen, if there are less than the counter limit per millisecond. While the degree of randomness involved otherwise is certainly extremely unlikely to have duplicates, I am a big fan of guarantees. Which is in fact why they put it into the RFC, as that is also a feature in many other popular id libraries. |
That said, "SHOULD" is indeed a recommendation, rather than requirement, and they list multiple alternatives. So I support having a separate set of ctor functions to call for the counter version(s). |
Hi Ashley, I'm Sergey Prokhorenko, a contributor to rfc4122bis and a counter enthusiast. Some developers implement the counter:
UUIDv7 with the counter for PostgreSQL is currently being developed in C by Andrey Borodin: But it seems to me that Rust is also a good tool for such development. |
We've currently actually got access to a counter in our
An alternative for 2. would be to add |
Like the vast majority, I am convinced that only the seventh version is worthy of attention. There is no need to waste time and effort implementing other versions.
I'm against. A counter initialized to zero increases the collision probability. Worse, the counter value may be the same as the random value used instead of the counter at the beginning of the millisecond. I prefer a counter that is initialized to a random value at the beginning of each millisecond. But this may reduce the actual capacity of the counter. Therefore, the leftmost bit of the counter should be initialized to zero and/or the timestamp should be incremented when the counter overflows.
The timestamp in the seventh version is shorter, and you missed the ver and var segments.
If we are talking about initializing the counter every millisecond with a random value and incrementing within a millisecond, then I am for it. |
This library already implements |
I highly recommend looking at the UUIDv7 implementation in PostgreSQL v17 by Andrey Borodin |
Thanks @sergeyprokhorenko 👍 Having a good reference implementation to point at will definitely be helpful |
Another good implementation: It is mentioned in this benchmark. Due to the excessively long counter (initialized with a random number every millisecond), less entropy is generated, which requires a lot of resources. But it's worth trying to replace rand::rngs::OsRng with openssl-rand for better performance. |
This patch has already been successfully tested: https://ardentperf.com/2024/02/03/uuid-benchmark-war/ |
Thought/Question: Why not add a builder pattern with defaults? That doesn't eliminate the backwards compatibility and can improve clarity around generating the UUID. |
The API in the Rust ULID crate provides monotonicity using an If you know that you are generating a batch then an Here's is an example: use ulid::Ulid;
use uuid::Uuid;
fn main() {
let mut uuids = Vec::with_capacity(10);
for _ in 0..10 {
let uuid = Uuid::now_v7();
uuids.push(uuid);
}
for uuid in uuids.iter() {
println!("{:<12} {uuid}", "uuidv7:");
}
println!("");
let mut ulids = Vec::with_capacity(10);
let mut ulid = Ulid::new();
for _ in 0..10 {
ulids.push(ulid);
ulid = ulid.increment().unwrap();
}
for ulid in ulids.iter() {
println!("{:<12} {ulid}", "ulid:");
}
} Is taking blatant inspiration from the excellent work in the ULID crate ok? Of course. ULID is the very first referred to time-based unique ID referenced in the new RFC. Note that this also gives more control over creation of batches. If one creates a batch of 1000 UUIDs, using an internal counter you have no control over whether they will be in the same millisecond. This means you will have part of the batch spread across different timestamps and the random component will jump around for each millisecond. Using an increment you can keep the millisecond and the random component stable. It's not required in the specification but seems useful to be able to trivially see if UUIDs are part of the same batch. |
This is not true |
New functions for generating UUIDv7 with a counter in the ClickHouse DBMS: link The structure is exactly the same as https://crates.io/crates/uuid7 and https://www.npmjs.com/package/uuidv7 |
I've started working on an implementation over in #755 that supports respecting the counter value when constructing v7 UUIDs in a way that lets a caller decide how wide they want that counter to be so the guarantees about ordering and uniqueness are configurable. It's still just a draft so will ping once it's ready. |
How might you allow the caller to control whether the UUIDs are generated for the same millisecond if the timer is internal to the uuid generator? |
UUIDs generated in different milliseconds have different timestamp values. UUIDs generated at the same millisecond have the same timestamp values. |
RFC9562 "Universally Unique IDentifiers (UUID)" has already been published and entered into force. Differences between the implementation for DBMS ClickHouse and the implementation for DBMS PostgreSQL:
In both implementations in case the counter overflows, the timestamp field is incremented by 1 and the counter is reset to a random new start value (guard bit to zero). I would also recommend to add the ability to shift the timestamp by the value specified by the formal parameter
This is allowed by RFC9562:
It is quite difficult to implement the exclusion (in the unpredictable future) of leap seconds required by RFC9562:
|
In order to maintain the invariant that UUIDs are sortable by the order they’re generated in this would need to increment the timestamp for all subsequently generated UUIDs in that millisecond, right? |
Yes that's right. Moreover, this may continue for several consecutive milliseconds of high generation rate until the real time catches up with the timestamp. But with a guard bit and a counter 18 bits or longer, a counter overflow is an impossible event (until a quantum computer came along). |
UUIDv7 extension written in RUST, implemented in PostgreSQL |
All of these calculations make the mistaken implicit assumption that all generated UUIDs end up in a common database table where they can collide. The absurdity of these calculations can be confirmed by the following example: UUIDv7s with a counter generated by one generator will never collide, even if random parts are completely removed from them. This is just as true as integer keys generated by autoincrement do not collide. The random part is necessary for the uniqueness of UUIDs generated by different generators (when merging database tables or at distributed UUID generation). By the way, the counter is initialized with a random number every millisecond. Therefore, it is incorrect to say that the counter reduces the random part. |
The authors of RFC 9562 did not intend to reject any of the contributors' proposals, but simply to combine all of these proposals. This does not mean that all these proposals were justified. But in real implementations Method 1 is used, which is not at all divided into two opposing strategies, but is single. Your proposal differs only in the absence of the random part generated for each individual UUID. It kills the unguessability without giving anything in return. |
The authors and contributors of RFC 9562 have detailed knowledge of monotonic ULIDs. Moreover, monotonic ULIDs were the basis for the development of UUIDv7. However, authors and contributors clearly saw the disadvantages of monotonic ULIDs. To address these disadvantages, UUIDv7 added a counter that is initialized with a random number and is protected from overflow. Adding a counter didn't make anything worse. It seems to me that there is no point in continuing this discussion, because you are not making valid arguments. |
Discussion about the two formal parameters of the function that generates UUIDv7: ClickHouse/ClickHouse#62852 (comment) |
In either case, |
I've gotten around to implementing a counter for version 7 UUIDs based on the recommendations here: https://github.com/uuid-rs/uuid/pull/755/files#diff-df69eeff48c2663ed59b31541c6c396ffecfed985b395daadafda019a524f75cR627-R674 EDIT: Looks like GitHub is collapsing the relevant part of the diff, so I'll just drop it in here: fn generate_timestamp_sequence(
&self,
seconds: u64,
subsec_nanos: u32,
) -> (Self::Output, u64, u32) {
use std::time::Duration;
let millis = (seconds * 1000).saturating_add(subsec_nanos as u64 / 1_000_000);
let last_reseed = self.last_reseed.get();
// If the observed system time has shifted forwards then regenerate the counter
if millis > last_reseed {
// Leave the most significant bit unset
// This guarantees the counter has at least 2,199,023,255,552
// values before it will overflow, which is exceptionally unlikely
// even in the worst case
let counter = crate::rng::u64() & (u64::MAX >> 23);
self.counter.set(counter);
self.last_reseed.set(millis);
(counter, seconds, subsec_nanos)
}
// If the observed system time has not shifted forwards then increment the counter
else {
// If the incoming timestamp is earlier than the last observed one then
// use it instead. This may happen if the system clock jitters, or if the counter
// has wrapped and the timestamp is artificially incremented
let millis = last_reseed;
// Guaranteed to never overflow u64
let counter = self.counter.get() + 1;
// If the counter has not overflowed its 42-bit storage then return it
if counter <= u64::MAX >> 22 {
self.counter.set(counter);
(counter, seconds, subsec_nanos)
}
// Unlikely: If the counter has overflowed its 42-bit storage then wrap it
// and increment the timestamp. Until the observed system time shifts past
// this incremented value, all timestamps will use it to maintain monotonicity
else {
let counter = 0;
self.counter.set(counter);
self.last_reseed.set(millis + 1);
let new_ts =
Duration::new(seconds, subsec_nanos) + Duration::from_millis(1);
(counter, new_ts.as_secs(), new_ts.subsec_nanos())
}
}
} I'd appreciate a review of the parameters to make sure they're reasonable to most users. The counter:
|
I would recommend:
|
Thanks @sergeyprokhorenko. We unfortunately can’t directly do 2. or 3. because of backwards compatibility we can’t add or remove function parameters or change function names, but I can use a global lock internally. You should still be able to shift timestamps when constructing UUIDs yourself through I also noticed it was pointed out other implementations reset the counter to a random value instead of zero on overflow so can make that change too. |
I've merged #755. It needs a bit of a write-up, but the final result should be in-line with the parameters set out earlier in this thread. We guarantee that any UUID produced by |
This is now supported as of |
Per the new draft "With this method rand_a section of UUIDv7 SHOULD be utilized as fixed-length dedicated counter bits that are incremented by one for every UUID generation."
That's in the "Fixed-Length Dedicated Counter Bits (Method 1)" section under https://datatracker.ietf.org/doc/html/draft-peabody-dispatch-new-uuid-format-04#monotonicity_counters which is directly linked from https://datatracker.ietf.org/doc/html/draft-peabody-dispatch-new-uuid-format-04#section-5.2 under the rand_a explanation.
To my reading, that means it should have a counter, like v6 (and v1), but instead the current v7 builder implementation https://docs.rs/uuid/latest/src/uuid/v7.rs.html#47-53 is just random for rand_a. Am I missing something on this one? If not, and this is a desirable feature, I might look into putting up a PR to add the counter support.
The text was updated successfully, but these errors were encountered: