Skip to content

release-20.1: vendor: Bump pebble to 6c5e6ce917e124c3d8b50387b998686dc3766dae#50282

Merged
itsbilal merged 1 commit intocockroachdb:release-20.1from
itsbilal:bump-pebble-20.1
Jun 16, 2020
Merged

release-20.1: vendor: Bump pebble to 6c5e6ce917e124c3d8b50387b998686dc3766dae#50282
itsbilal merged 1 commit intocockroachdb:release-20.1from
itsbilal:bump-pebble-20.1

Conversation

@itsbilal
Copy link
Copy Markdown
Contributor

@itsbilal itsbilal commented Jun 16, 2020

Changes pulled in:

  • 25e83e93536da50490a4ff90400053f2ae3217e1 db: wake all waiters on cleaner.cond on state transitions
  • 6681936b9aa427e7c4f5a8405d634392e0a37aa8 internal/record: Handle ErrUnexpectedEOFs like invalid records

Release note (bug fix): Fix a bug where restarting cockroach with
the pebble storage engine after a crash during write-ahead logging
could, in some rare cases, return an "unexpected EOF" error.

@itsbilal itsbilal requested a review from petermattis June 16, 2020 16:24
@itsbilal itsbilal self-assigned this Jun 16, 2020
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Collaborator

@petermattis petermattis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Changes pulled in:
 - 25e83e93536da50490a4ff90400053f2ae3217e1 db: wake all waiters on cleaner.cond on state transitions
 - 6681936b9aa427e7c4f5a8405d634392e0a37aa8 internal/record: Handle ErrUnexpectedEOFs like invalid records

Release note (bug fix): Fix a bug where restarting cockroach with
the pebble storage engine after a crash during write-ahead logging
could, in some rare cases, return an "unexpected EOF" error.
@itsbilal
Copy link
Copy Markdown
Contributor Author

TFTR!

@itsbilal itsbilal merged commit 1217bdc into cockroachdb:release-20.1 Jun 16, 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

Successfully merging this pull request may close these issues.

3 participants