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 field analyzers loading when search changes #3049

Merged
merged 1 commit into from Nov 2, 2016

Conversation

Projects
None yet
2 participants
@edmundoa
Member

edmundoa commented Nov 2, 2016

Closes #3030

(cherry picked from commit 277af69)

Fix field analyzers loading when search changes
* Fix search params propagated to field analyzers

Before this change we were using the current input values as
search parameters, instead of using those used in the last executed
search.

* Update field analyzers when search changes

When a new search is executed, fetch new quick values and field
statictics data. In this way the displayed information will be in sync
with the search query, time range, and stream used in the message list
and histogram.

The field graphs were already updating when the search histogram
changed, so they do not require further changes at this point.

Closes #3030

(cherry picked from commit 277af69)

@edmundoa edmundoa added this to the 2.1.2 milestone Nov 2, 2016

@joschi

joschi approved these changes Nov 2, 2016

@joschi joschi merged commit 35d8385 into 2.1 Nov 2, 2016

5 checks passed

ci-server-integration Jenkins build graylog2-server-integration-pr 1516 has succeeded
Details
ci-web-linter Jenkins build graylog-pr-linter-check 998 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

@joschi joschi deleted the issue-3030 branch Nov 2, 2016

@joschi joschi removed the ready-for-review label Nov 2, 2016

@joschi joschi self-assigned this Nov 2, 2016

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