-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Event UUIDs are not unique #30692
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
Comments
Hey @isAdrisal , Thanks much for the bug report! I noticed that the duplicated IDs are on events captured with Could you please try updating to the latest/current version v3.12.0, and then check to see if you're still seeing duplicated event UUIDs in events captured with the latest version? Thanks! |
Hi @slshults was there a specific known issue that was resolved in later versions of that library? Just wanting to understand if the upgrade is a likely fix or if we're trying to eliminate a variable as these upgrades typically have some additional work associated with them. |
Hey @isAdrisal , That's an understandable question, good call. There have been several capture-related changes which could impact what you're seeing, but not specifically addressing UUIDs being assigned to multiple events. Would you be able to search your code for Please let me know if find that you're not assigning UUIDs, and it's being handled only by our SDK. If that's the case, then I'll get this bug report into a queue for engineers to look at asap. Thanks! |
Hi @slshults, thanks for the extra detail! I've checked through our codebase and we're not setting I did a little more digging to try and narrow down the issue. It looks like all of the duplicate uuids are specifically from There was a change to use uuidv7 early last year (timeline below), but I suspect that's not the smoking gun it may have looked like given we were also using that uuidv7 change before upgrading to 3.1.1 Timeline:
I'll still look at doing the upgrade to |
Thanks much @isAdrisal , I've added some labels to this issue, and sent a heads-up to our product analytics engineers! |
@slshults an update — we're still seeing duplicate event UUIDs even after upgrading to posthog-react-native 3.12.0. We're also seeing the issue on other libraries too, like posthog-go and the web SDK. We are not manually setting UUIDs with any of these libraries. |
Is it possible this is just a documentation issue? Are event UUIDs not globally unique, but only unique in a certain scope — like per "event" type? |
Bug Description
Events in PostHog (Cloud) have a
uuid
field. Presumably, this is intended to be a unique value. However, at least in our instance it's not.Some rough calculations show that around 0.7% of our event UUIDs appear more than once with other events.
Practically, this means when joining on
uuid
only in sql queries, duplicate rows are returned where they shouldn't.How to reproduce
See screenshots, or run this query.
Additional context
Debug info
Session: https://us.posthog.com/project/sTMFPsFhdP1Ssg/replay/0195f5c1-8d84-7fd9-8dfe-7c98353c6079?t=10777 Admin: http://go/adminOrgUS/018a1aa0-fba1-0000-7c63-3525ad2656c8 (project ID 34169) Sentry: http://go/sentryUS/34169
The text was updated successfully, but these errors were encountered: