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

Stress error utf 8 codec cant decode byte #498

Merged
merged 1 commit into from
May 11, 2023

Conversation

Yuval-Ariel
Copy link
Contributor

closes #496

@Yuval-Ariel Yuval-Ariel added Upstreamable can be upstreamed to RocksDB bug fix Fixes a known bug labels May 8, 2023
@Yuval-Ariel Yuval-Ariel requested a review from udi-speedb May 8, 2023 09:34
@Yuval-Ariel Yuval-Ariel self-assigned this May 8, 2023
@Yuval-Ariel Yuval-Ariel changed the base branch from main to 8-1-1-rebase-main-candidate May 8, 2023 09:34
udi-speedb
udi-speedb previously approved these changes May 8, 2023
@udi-speedb
Copy link
Contributor

@Yuval-Ariel - Please just verify that no other call to this function suffers from the same problem.
Please also mark the issue as upstreamable as this is a bug in RocksDB

@Yuval-Ariel
Copy link
Contributor Author

done. i'll merge it to main once the rebase is over.

@Yuval-Ariel Yuval-Ariel force-pushed the 8-1-1-rebase-main-candidate branch from 3349f79 to 2ffee5b Compare May 11, 2023 05:43
@Yuval-Ariel Yuval-Ariel changed the base branch from 8-1-1-rebase-main-candidate to main May 11, 2023 05:57
@Yuval-Ariel Yuval-Ariel dismissed udi-speedb’s stale review May 11, 2023 05:57

The base branch was changed.

@Yuval-Ariel Yuval-Ariel force-pushed the stress-error-utf-8-codec-cant-decode-byte branch from 407f6c1 to b3db7cd Compare May 11, 2023 05:58
@Yuval-Ariel
Copy link
Contributor Author

@udi-speedb plz reapprove. i've rebased on current main

@udi-speedb udi-speedb self-requested a review May 11, 2023 07:08
@Yuval-Ariel Yuval-Ariel merged commit 9cee48e into main May 11, 2023
@Yuval-Ariel Yuval-Ariel deleted the stress-error-utf-8-codec-cant-decode-byte branch May 11, 2023 12:06
udi-speedb pushed a commit that referenced this pull request Nov 19, 2023
when verify_before_write = true, msg_prefix was misplaced and the value from the db was inserted instead of it which led to junk and a decoding error.
udi-speedb pushed a commit that referenced this pull request Dec 5, 2023
when verify_before_write = true, msg_prefix was misplaced and the value from the db was inserted instead of it which led to junk and a decoding error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Fixes a known bug Upstreamable can be upstreamed to RocksDB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stress error: 'utf-8' codec can't decode byte 0xe3
2 participants