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

Full data integrity check draft #75

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

isapego
Copy link

@isapego isapego commented Sep 28, 2024

The main idea: We scan over all the documents, hashing them, multiplying hashes (order-independent check, xor can be used as an alternative) and then using the same logic as in basic verification to combine the resulting hashes with the respecting namespaces and get the final hash.

The cons of this approach -- it takes the same amount of time as the reading of the whole DB. There are different ways that we could use to improve the performance (e.g. hashing on the server side using js functions), but this approach is not universal and should be tested across different DBMS separately. Probably, an optimization, that can be implemented in the future.

The code is a draft, which means:

  • There are no tests;
  • A lot of boilerplate (was planning to clean it later);
  • The command line key is worded horribly (--verify-fully).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant