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

Revert removal of DashboardRegistry in Graylog 1.3.x #1681

Merged
merged 1 commit into from Jan 11, 2016
Merged

Conversation

@joschi
Copy link
Contributor

@joschi joschi commented Jan 8, 2016

This reverts commit 8336a8d and f997da6. The removal of DashboardRegistry effectively disabled the caching of the computation results of dashboard widgets so that every request would access Elasticsearch to calculate the widget's value.

Fixes #1640, refs #1588, refs #1542

Jochen Schalanda
This reverts commit 8336a8d and f997da6.
Fixes #1640, refs #1588, refs #1542
@joschi joschi added this to the 1.3.3 milestone Jan 8, 2016
@dennisoelkers dennisoelkers self-assigned this Jan 8, 2016
dennisoelkers added a commit that referenced this pull request Jan 11, 2016
Revert removal of DashboardRegistry in Graylog 1.3.x
@dennisoelkers dennisoelkers merged commit ebaf8c4 into 1.3 Jan 11, 2016
2 of 3 checks passed
2 of 3 checks passed
ci Jenkins build graylog2-server-integration-pr 491 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
@dennisoelkers dennisoelkers deleted the issue-1640 branch Jan 11, 2016
leusonmario pushed a commit to leusonmario/graylog2-server that referenced this pull request Aug 2, 2017
Fixes Graylog2#1681

(cherry picked from commit 27c5c18c1d5c5cb08d5070b4a2f7df5a5d366739)
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
You can’t perform that action at this time.