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

Drop discard stats if we can't unmarshal it #936

Merged
merged 2 commits into from
Jul 18, 2019

Conversation

jarifibrahim
Copy link
Contributor

@jarifibrahim jarifibrahim commented Jul 17, 2019

We don't want to prevent badger from starting if we can't read/unmarshal
discard stats.

Fixes hypermodeinc/dgraph#3669


This change is Reviewable

We don't want to prevent badger from starting if we can't read/unmarshal
discard stats.
@jarifibrahim jarifibrahim requested a review from a team July 17, 2019 19:38
Copy link
Contributor

@manishrjain manishrjain left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 2 of 2 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ashish-goswami)

@jarifibrahim jarifibrahim merged commit c5bf7e3 into master Jul 18, 2019
@jarifibrahim jarifibrahim deleted the ibrahim/discardstats-truncated branch July 18, 2019 09:27
jarifibrahim pushed a commit that referenced this pull request Oct 24, 2019
We don't want to prevent badger from starting up if we can't
read/unmarshal discard stats.
jarifibrahim pushed a commit that referenced this pull request Mar 12, 2020
* Drop discard stats if we can't unmarshal it

We don't want to prevent badger from starting up if we can't
read/unmarshal discard stats.

(cherry picked from commit c5bf7e3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Error while creating badger KV posting store error: unexpected end of JSON input
3 participants