-
Notifications
You must be signed in to change notification settings - Fork 232
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
Consider just using unique IDs instead of UUIDs #50
Comments
I don't believe the spec specifies that |
@asadowns Thanks for the response! As I read it, there is a UUID to identify vehicles in the agency API, but it's called |
Ahh. good point. I guess it's clarified a bit on the
|
@asadowns your interpretation of the
More context into the decision to use both can be had in #32; As for whether UUID or string is best for these identifiers meant for machines, I think that is still up for debate. |
We very quickly noticed performance problems in our data warehouse when we introduced having to index on solely a string ( Having a UUID allows us to efficiently aggregate data from across many providers, this stack overflow post was helpful for me in understanding why we should get UUID. Which is that we are generating a distributed index across providers. |
From an implementer's perspective, I'd argue that requiring that vehicles, devices, and trips all have UUIDs could be a bit of pain, and I'm not totally sure why it's required. Since a provider is presumably in charge of all their fleet data, they would have the ability to allocate unique IDs without using UUID. What do you think about just changing this to String type and requiring that it be unique by provider?
The text was updated successfully, but these errors were encountered: