Skip to content
This repository has been archived by the owner on Apr 29, 2020. It is now read-only.

resource name nbs are too short #17

Open
osrf-migration opened this issue Feb 16, 2017 · 0 comments
Open

resource name nbs are too short #17

osrf-migration opened this issue Feb 16, 2017 · 0 comments
Labels
bug Something isn't working major

Comments

@osrf-migration
Copy link

Original report (archived issue) by Hugo Boyer (Bitbucket: hugomatic, GitHub: hugomatic).


3 characters is small, and will rollover soon.
string compare won't guarantee order.

A better fix would be to split resource name into:

an id (could be a uuid)
a creation time (for sorting, unless the id has increment part)
a type ("simulator", etc)

The current scheme requires to call the database INCR to generate a unique resource name. This operation is asynchronous and could be avoided.

@osrf-migration osrf-migration added major bug Something isn't working labels Apr 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working major
Projects
None yet
Development

No branches or pull requests

1 participant