Register metrics defined at Resource level #1105

Merged
merged 1 commit into from Mar 19, 2017

Conversation

Projects
None yet
3 participants
@arteam
Member

arteam commented Mar 14, 2017

Backport of #920

If a Resource class is annotated with either:

  • @Timed,
  • @Metered
  • @ExceptionMetered

all method defined in such class will 'inherit'
the corresponding annotation.

Register metrics defined at Resource level
If a `Resource` class is annotated with either:

* `@timed`,
* `@Metered`
* `@ExceptionMetered`

all method defined in such class will 'inherit'
the corresponding annotation.
@arteam

This comment has been minimized.

Show comment
Hide comment
@arteam

arteam Mar 14, 2017

Member

@jplock, @patrox: Please review.

Member

arteam commented Mar 14, 2017

@jplock, @patrox: Please review.

@jplock jplock added this to the 3.2.2 milestone Mar 19, 2017

@jplock jplock added the feature label Mar 19, 2017

@jplock

jplock approved these changes Mar 19, 2017

@jplock jplock merged commit 16b825e into 3.2-development Mar 19, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@jplock jplock deleted the jersey_resource_metrics_backport branch Mar 19, 2017

@patrox

This comment has been minimized.

Show comment
Hide comment
@patrox

patrox Mar 21, 2017

Contributor

@arteam, @jplock Sorry, I was away for few days, enjoying some short vacations and St. Patrick's day.
Thanks for pulling it in !

Contributor

patrox commented Mar 21, 2017

@arteam, @jplock Sorry, I was away for few days, enjoying some short vacations and St. Patrick's day.
Thanks for pulling it in !

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