Implementation of Seguro Phase II

wexpertsystems / seguro

Distributed event log storage for Urbit.

MIT License

5 stars 2 forks source link

Implementation of Seguro Phase II #1

Closed ashelkovnykov closed 2 years ago

ashelkovnykov commented 2 years ago

Includes:

Seguro Phase II
Unit tests
Integration tests
Benchmark suite
Benchmark results
makefile
Fully commented
Enhanced README

Notes: A partially-complete version of Seguro Phase II that follows the design outlined in the proposal more closely is available on a different branch in my fork. It is trivial to git cherry-pick or git rebase the two branches together. However, I think the version included in this PR is the one we should go with.

matthew-levan commented 2 years ago

Looks good. Only a few more questions above, then I'll approve and merge. Thank you.

ashelkovnykov commented 2 years ago

GH isn't showing it well, but I pushed a new version of the final commit. It fixed the typo for "129 fragments or more", fixed inconsistent usage of "additional"/"total" to "remaining", and added a little more explanation to the fragment header.