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

Allow metrics collection #30

Closed
masterspambot opened this issue Jul 29, 2016 · 9 comments
Closed

Allow metrics collection #30

masterspambot opened this issue Jul 29, 2016 · 9 comments

Comments

@masterspambot
Copy link
Member

From @meshuga on April 21, 2016 20:7

Add metrics functionality for Katharsis. It should be possible to integrate with web framework functionalities e.g. Dropwizard/Jersey metrics (https://dropwizard.github.io/metrics/3.1.0/apidocs/com/codahale/metrics/jersey2/InstrumentedResourceMethodApplicationListener.html) or Spring metrics (https://docs.spring.io/spring-boot/docs/current/reference/html/production-ready-metrics.html). It should be possible to gather infomrations about requests and exceptions.
Plus add docs on how to do that.

Copied from original issue: katharsis-project/katharsis-core#299

@masterspambot
Copy link
Member Author

From @ieugen on May 18, 2016 11:49

We also need this and I going to do some work to add metrics. Please publish your work on a branch/project if you have something.

@masterspambot
Copy link
Member Author

From @ieugen on May 25, 2016 9:52

After the refactoring I'm going to work with https://github.com/astefanutti/metrics-aspectj to produce something useful and come back with results. I thing it's the least intrusive way of doing it.

@masterspambot
Copy link
Member Author

From @remmeier on June 29, 2016 18:30

+1 to use dropwizard metrics, is quite popular with already many other plugins

@remmeier
Copy link
Contributor

remmeier commented Aug 1, 2016

next to dropwizard metrics zipkin/brave might also be worth investigating.

As part of with #45 there is a request filter api similar to the one for servlets and jaxrs. That might be of help here too. And the metric part could then be implemented as a "module"/add-on to avoid additional dependencies in core.

@derekbassett
Copy link

Any idea if this is possible in Katharsis 3? If I submitted a PR related to this would it be considered for being accepted?

@chb0github
Copy link
Contributor

What metrics did you have in mind?

On Thu, Nov 3, 2016, 3:39 PM Derek Bassett notifications@github.com wrote:

Any idea if this is possible in Katharsis 3? If I submitted a PR related
to this would it be considered for being accepted?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#30 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABaI0FVOlstMqE9O1CdqEgLRJhP0iv6Tks5q6mKogaJpZM4JYH7T
.

@remmeier
Copy link
Contributor

remmeier commented Nov 4, 2016

note that tracing is also on its way: #192

remmeier added a commit that referenced this issue Nov 5, 2016
new RepositoryFilter interface introduced that sits in the repository
layer and allows to intercept request, respones and links/meta
information gathering.

- repository adapters modified to invoke filters
- filters get a chain class to decide whether to proceed with the
request or return a result on its own.
- RepositoryRequestSpec to specify the contents of a request
- existing Filter interface should be renamed to make clear that they
operate on a different layer (transport vs repository).
- Module api extended to allow RepositoryFilter registration
- minor refactoring to ResourceRegistry/ModuleRegistry dependency to
allow access to RepositoryFilter
remmeier added a commit that referenced this issue Nov 5, 2016
#30 RepositoryFilter to allow metrics collection
@remmeier
Copy link
Contributor

remmeier commented Nov 5, 2016

A new interface "RepositoryFilter" is available that can be registered with the modules. That should help quite a bit if somebody would like to implement a metrics module.

@chb0github
Copy link
Contributor

Closing: There are any of several ways to do metrics on resources. Katharsis is not in that business.

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

No branches or pull requests

4 participants