Skip to content
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

Support binary uuid #5

Closed
vaishnavmhetre opened this issue Jun 26, 2019 · 3 comments
Closed

Support binary uuid #5

vaishnavmhetre opened this issue Jun 26, 2019 · 3 comments
Labels
wontfix This will not be worked on

Comments

@vaishnavmhetre
Copy link

String comparison of UUID is least recommended due to infamous cycles of comparison as compared to integer.
Support article - Optimised UUID Storage

Currently recommended repository Laravel Efficient UUID be referred more of to get an idea of implementation.

Although not many repositories yet support uuid alternative as key identification of data, hence above alternative repository seemed more helpful. (They also have a normal uuid repo too.)

I loved your ease of usage over their repo, hence suggesting their ideology and your accessibility, together, will be a big charm.

@matthew-inamdar
Copy link
Member

This is a really useful solution for high performance requirements. However this package aims to be a simple solution for all. From what I can understand, the binary() column only supports a subset of the databases Eloquent supports?

@matthew-inamdar matthew-inamdar added enhancement New feature or request question Further information is requested labels Jun 27, 2019
@matthew-inamdar
Copy link
Member

@vaishnavmhetre marking this as wontfix due to not having enough information. If you have any input then please reopen.

@matthew-inamdar matthew-inamdar added wontfix This will not be worked on and removed enhancement New feature or request question Further information is requested labels Jul 2, 2019
@blite
Copy link

blite commented Dec 6, 2021

should be on the readme as the index grows large extremely quickly with string types

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Development

No branches or pull requests

3 participants