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

Always include index range of deflector target index #1679

Merged
merged 1 commit into from Jan 11, 2016

Conversation

Projects
None yet
2 participants
@joschi
Contributor

joschi commented Jan 8, 2016

The index range of the deflector target index can't be calculated until the index has been rotated, so a dummy range is being used. For this reason, the latest index always has to be included in searches and not only in the case of searches with relative time range.

Fixes #1672

Always include index range of deflector target index
The index range of the deflector target index can't be calculated until the index
has been rotated, so a dummy range is being used. For this reason, the latest index
*always* has to be included in searches and not only in the case of searches with
relative time range.

Fixes #1672

@joschi joschi modified the milestones: 1.x, 1.3.3 Jan 8, 2016

@bernd bernd self-assigned this Jan 11, 2016

@bernd

This comment has been minimized.

Member

bernd commented Jan 11, 2016

LGTM 👍

bernd added a commit that referenced this pull request Jan 11, 2016

Merge pull request #1679 from Graylog2/issue-1672
Always include index range of deflector target index

@bernd bernd merged commit c776924 into 1.3 Jan 11, 2016

2 of 3 checks passed

ci Jenkins build graylog2-server-integration-pr 485 has failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@bernd bernd deleted the issue-1672 branch Jan 11, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment