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

Minor lookup table fixes #4266

Merged
merged 3 commits into from Oct 19, 2017
Merged

Minor lookup table fixes #4266

merged 3 commits into from Oct 19, 2017

Conversation

@kroepke
Copy link
Member

@kroepke kroepke commented Oct 18, 2017

The DSV HTTP adapter used the wrong component in the frontend and its internal fields were a bit complicated.

Fix a gauge metric reregistration issue in LookupCaches.

kroepke added 2 commits Oct 17, 2017
Otherwise restarting a cache throws an exception is the lookup table fails.
@kroepke kroepke added this to the 2.4.0 milestone Oct 18, 2017
@kroepke kroepke requested a review from bernd Oct 18, 2017
@ghost ghost assigned kroepke Oct 18, 2017
@bernd bernd assigned bernd and unassigned kroepke Oct 19, 2017
@bernd
bernd approved these changes Oct 19, 2017
@bernd bernd merged commit 13855de into master Oct 19, 2017
5 checks passed
5 checks passed
@garybot2
ci-web-linter Jenkins build graylog-pr-linter-check 2017 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 623 has succeeded
Details
license/cla Contributor License Agreement is signed.
Details
@bernd bernd deleted the cleanup-dsv branch Oct 19, 2017
bernd added a commit that referenced this pull request Oct 19, 2017
* use correct field set component for DSV-HTTP adapter

* safely register eventcount gauge

Otherwise restarting a cache throws an exception is the lookup table fails.

* fix license header format

(cherry picked from commit 13855de)
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