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

Use LUID's instead of Random UUID's #338

Merged
merged 1 commit into from
Mar 4, 2021
Merged

Use LUID's instead of Random UUID's #338

merged 1 commit into from
Mar 4, 2021

Conversation

alexanderkiel
Copy link
Member

Random UUID's are expensive to create. LUID's are much faster. The lower
entropy of 36 bit is not a problem, because not more than one bundle
should be generated for each client per millisecond.

Random UUID's are expensive to create. LUID's are much faster. The lower
entropy of 36 bit is not a problem, because not more than one bundle
should be generated for each client per millisecond.
@alexanderkiel alexanderkiel self-assigned this Mar 4, 2021
@alexanderkiel alexanderkiel added this to the v0.11 milestone Mar 4, 2021
@alexanderkiel alexanderkiel added the performance Performance improvement label Mar 4, 2021
@codecov
Copy link

codecov bot commented Mar 4, 2021

Codecov Report

Merging #338 (d0b02c8) into develop (fe731e2) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #338   +/-   ##
========================================
  Coverage    88.38%   88.38%           
========================================
  Files          200      200           
  Lines        12419    12419           
  Branches       466      466           
========================================
  Hits         10976    10976           
  Misses         977      977           
  Partials       466      466           
Impacted Files Coverage Δ
...raction/src/blaze/interaction/history/instance.clj 93.61% <100.00%> (ø)
...teraction/src/blaze/interaction/history/system.clj 96.00% <100.00%> (ø)
...interaction/src/blaze/interaction/history/type.clj 86.66% <100.00%> (ø)
...ction/src/blaze/interaction/search_compartment.clj 85.26% <100.00%> (ø)
...nteraction/src/blaze/interaction/search_system.clj 87.64% <100.00%> (ø)
.../interaction/src/blaze/interaction/search_type.clj 92.24% <100.00%> (ø)
.../interaction/src/blaze/interaction/transaction.clj 95.58% <100.00%> (ø)

@alexanderkiel alexanderkiel merged commit ce1671f into develop Mar 4, 2021
@alexanderkiel alexanderkiel deleted the luid branch March 4, 2021 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant