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

Fix number parsing in Indices#getStoreSiztInBytes() #4008

Merged
merged 1 commit into from Jul 17, 2017
Merged

Fix number parsing in Indices#getStoreSiztInBytes() #4008

merged 1 commit into from Jul 17, 2017

Conversation

@bernd
Copy link
Member

@bernd bernd commented Jul 17, 2017

The size might be an integer instead of a long. Unbreaks the index size rotation strategy.

Fixes #3997

The size might be an integer instead of a long.

Fixes #3997
@bernd bernd requested a review from joschi Jul 17, 2017
@bernd
Copy link
Member Author

@bernd bernd commented Jul 17, 2017

This needs to go into 2.3 as well.

@joschi
joschi approved these changes Jul 17, 2017
@joschi joschi self-assigned this Jul 17, 2017
@joschi joschi merged commit b29b103 into master Jul 17, 2017
4 checks passed
4 checks passed
@garybot2
ci-web-linter Jenkins build graylog-pr-linter-check 1810 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
@garybot2
graylog-project/pr Jenkins build graylog-project-pr-snapshot 321 has succeeded
Details
@joschi joschi deleted the issue-3997 branch Jul 17, 2017
bernd added a commit that referenced this pull request Jul 17, 2017
The size might be an integer instead of a long

Fixes #3997
(cherry picked from commit b29b103)
joschi added a commit that referenced this pull request Jul 17, 2017
The size might be an integer instead of a long

Refs #4008
Fixes #3997
(cherry picked from commit b29b103)
mariussturm added a commit that referenced this pull request Jul 18, 2017
The size might be an integer instead of a long

Fixes #3997
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.

None yet

2 participants