-
Notifications
You must be signed in to change notification settings - Fork 23
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
【Feature】add crc to wal #407
Comments
Yes, one idea was to use a crc based on the past entries of the log as well. (eg: similar to git history). That would help in identifying any issues in non-common histories of the wal across nodes. |
/assign |
@lsytj0413 Perhaps we can start with a proposal and design doc for how to implement the change. We can use discussions. eg: https://github.com/streamnative/oxia/discussions/new?category=proposal |
Yes, i will open an proposal to describe the detail design first. |
Propocal at #412 ,PTAL |
oxia use wal to storage the
LogEntry
in segment files, should we addcrc
to each record to protect data corrupted?The text was updated successfully, but these errors were encountered: