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

Add timing metrics to GelfOutput #3810

Merged
merged 1 commit into from May 16, 2017

Conversation

Projects
None yet
2 participants
@joschi
Contributor

joschi commented May 11, 2017

Types of changes

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Closes #3716

@joschi joschi added this to the 2.3.0 milestone May 11, 2017

@kroepke kroepke self-requested a review May 16, 2017

@kroepke

lgtm

@kroepke kroepke merged commit ff1a9f3 into master May 16, 2017

4 checks passed

ci-web-linter Jenkins build graylog-pr-linter-check 1632 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
graylog-project/pr Jenkins build graylog-project-pr-snapshot 100 has succeeded
Details

@kroepke kroepke deleted the issue-3716 branch May 16, 2017

bernd added a commit that referenced this pull request Jun 6, 2017

Revert "Add timing metrics to GelfOutput (#3810)"
This reverts commit ff1a9f3.

It broke backwards compatibility with 3rd party output plugins.

bernd added a commit that referenced this pull request Jun 6, 2017

Revert "Add timing metrics to GelfOutput (#3810)"
This reverts commit ff1a9f3.

It broke backwards compatibility with 3rd party output plugins.

Fixes #3851

kroepke added a commit that referenced this pull request Jun 6, 2017

Revert "Add timing metrics to GelfOutput (#3810)" (#3891)
This reverts commit ff1a9f3.

It broke backwards compatibility with 3rd party output plugins.

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