jmxtrans / embedded-jmxtrans

In process JMX metrics exporter.
MIT License
82 stars 37 forks source link

marked all of the OutputWriter implementations with a marker interface, for documentation #95

Closed YannRobert closed 9 years ago

YannRobert commented 9 years ago

Added 2 SOURCE level Annotations that I think may be useful for documentation. @ExplicitEpoch @ImplicitEpoch

I can think of future evolution where thoses Annotations could be retained at Runtime, in order to help JmxTrans to decide a rety-on-error strategy, or priorize export to those marked with ImplicitEpoch before those with ExplicitEpoch. If this PR is not a first step to this, it may at least serve as documentation.

Maybe the Annotations naming is not perfect. Please make suggestion.

What do you think of this?

buildhive commented 9 years ago

jmxtrans project » embedded-jmxtrans #110 SUCCESS This pull request looks good (what's this?)