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

log message "error loading snapshot" reported under simple workload #40

Open
mschoch opened this issue Dec 11, 2020 · 1 comment
Open

Comments

@mschoch
Copy link
Member

mschoch commented Dec 11, 2020

@rubiojr reports here #39 that when using a Reader not obtained from the Writer and while the Writer is still open, the message "error loading snapshot epoch: %d: %v" appears.

@mschoch
Copy link
Member Author

mschoch commented Dec 11, 2020

One theory is that as the Writer independently creates new snapshots, when opening a Reader (not via the Writer) it may attempt to open a partially written snapshot file. This would fail the CRC check, and another snapshot is tried.

We should be able to test this theory with a straightforward unit test.

One approach to fix this might be to write new snapshot files with a different extension, and rename them after a complete snapshot has been written. This would prevent new readers from seeing partially written snapshot files.

@mschoch mschoch mentioned this issue Dec 11, 2020
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

No branches or pull requests

1 participant