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

Mark message offset as committed in case of a decoding error #2643

Merged
merged 1 commit into from Aug 9, 2016
Merged

Conversation

@bernd
Copy link
Member

@bernd bernd commented Aug 9, 2016

This fixes an edge case where the journal grows when there is only one
input and no message can be decoded.

Fixes #2627

This fixes an edge case where the journal grows when there is only one
input and no message can be decoded.

Fixes #2627
@bernd bernd added this to the 2.1.0 milestone Aug 9, 2016
@joschi joschi self-assigned this Aug 9, 2016
@joschi
Copy link
Contributor

@joschi joschi commented Aug 9, 2016

LGTM. 👍

@joschi joschi merged commit 07fcf73 into master Aug 9, 2016
4 checks passed
4 checks passed
ci-server-integration Jenkins build graylog2-server-integration-pr 1230 has succeeded
Details
ci-web-linter Jenkins build graylog-pr-linter-check 713 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@joschi joschi deleted the issue-2627 branch Aug 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.