Binary primary key about rihanna HOT 3 CLOSED

@lleger thanks for opening an issue!

Yes, the migration uses integer primary keys. Can I ask why you would prefer a UUID?

from rihanna.

We use UUIDs as primary keys by default, so it would be nice to have all of the tables in the database use them consistently. If we add Rihanna, she would be the only table with a non-UUID primary key.

from rihanna.

It is not currently possible to customise the primary key in Rihanna.

I'm not sure I see any clear benefit to having the ID be a UUID other than consistency for consistency's sake in your case. It would come at a cost of some extra code complexity (and an extremely tiny performance hit) and would also interfere with the default sorting order - Rihanna finds the oldest job by sorting by (id, enqueued_at).

Unless there is some clear reason to do so I think I'd have to say I won't be able to support this.

You are welcome to fork the library and modify it yourself though :)

from rihanna.