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

Preparation work for corruption/recovery testing #27

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LorenzoBoccaccia
Copy link

  • changed upload/download format for saving one server round trip
  • added sector/device characteristics to prepare for recovery testing
  • added md5 page centric dump to make corruption visible in debug

first step if we want to test lock corruption would be to declare to sqlite the actual vfs invariants
the changes follow what I understand of the specs here https://www.sqlite.org/draft/fileio.html#tocentry_50

@michalc
Copy link
Member

michalc commented Apr 30, 2024

The diff looks very odd for some reason - like every single line has been changed, even thought that also clearly not the case. Do you know what's happened / if anything can be done to tidy it?

- changed upload/download format for saving one server round trip
- added sector/device characteristics to prepare for recovery testing
- added md5 page centric dump to make corruption visible in debug
@LorenzoBoccaccia
Copy link
Author

a few hiccups with white spaces

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

2 participants