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

Restrict search in RecentMessageLoader to 1 hour #3367

Merged
merged 2 commits into from Jan 17, 2017

Conversation

Projects
None yet
2 participants
@joschi
Contributor

joschi commented Jan 17, 2017

The RecentMessageLoader component used to query the complete available time range of all indices which can take very long on well populated Elasticsearch clusters.

This change restricts the queried time range to 1 hour (3600 s).

Fixes #3366

Restrict search in RecentMessageLoader to 1 hour
The RecentMessageLoader component used to query the complete available time range of all indices
which can take veeery long on well populated Elasticsearch clusters.

This change restricts the queried time range to 1 hour (3600 s).

Fixes #3366

@joschi joschi added this to the 2.2.0 milestone Jan 17, 2017

@dennisoelkers dennisoelkers self-assigned this Jan 17, 2017

@dennisoelkers

In general this is a valid way to fix this. It just prevents a corner case, where users have an input which receives data very seldomly, to be workable with the extractors. To help with this scenario, we should add the information that it is limited to going back one hour to the help text in the recent message loader component.

@dennisoelkers dennisoelkers merged commit a3d39be into master Jan 17, 2017

4 checks passed

ci-web-linter Jenkins build graylog-pr-linter-check 1286 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
licence/cla Contributor License Agreement is signed.
Details

@dennisoelkers dennisoelkers deleted the issue-3366 branch Jan 17, 2017

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