Hi Romain: Thanks for all the background information on this! I certainly appreciate it.
Mike. On Wed, Nov 28, 2018 at 10:47 AM Romain Manni-Bucau <[email protected]> wrote: > Hello Michael, > > in prometheus, histograms are not represented directly like their class > and the prometheus representation is done through an endpoint and not a > body writer as per microprofile spec - can likely be enhanced but it is not > needed today since you rarely want a single metrics in the output (see > https://github.com/apache/geronimo-metrics/blob/master/geronimo-metrics-common/src/main/java/org/apache/geronimo/microprofile/metrics/common/prometheus/PrometheusFormatter.java#L140 > for an impl) > > Now, geronimo-metrics provides for you the prometheus endpoints so you > don't need to implement it on your side, did you set the system property > geronimo.metrics.jaxrs.activated=true > ? > > Romain Manni-Bucau > @rmannibucau <https://twitter.com/rmannibucau> | Blog > <https://rmannibucau.metawerx.net/> | Old Blog > <http://rmannibucau.wordpress.com> | Github > <https://github.com/rmannibucau> | LinkedIn > <https://www.linkedin.com/in/rmannibucau> | Book > <https://www.packtpub.com/application-development/java-ee-8-high-performance> > > > Le mer. 28 nov. 2018 à 16:43, Ivan Junckes Filho <[email protected]> > a écrit : > >> Hi Michael, could you please push it to a remote branch so I can take a >> look? >> >> On Wed, Nov 28, 2018 at 1:26 PM Michael Redlich <[email protected]> >> wrote: >> >> > Hello everyone: >> > >> > I am making progress implementing a Metrics Histogram example. Please >> see >> > the following code: >> > >> > @Path("/histogram") >> > @GET >> > @Produces(MediaType.TEXT_PLAIN) >> > // @Produces(MediaType.APPLICATION_JSON) >> > public Histogram histogramStatus() { >> > Metadata metadata = new Metadata("items", MetricType.HISTOGRAM, >> > "degrees F"); >> > metadata.setDescription("A histogram of recent temperatures."); >> > Histogram temps = metrics.histogram(metadata); >> > for(int temp = 80; temp < 100; ++temp) { >> > temps.update(temp); >> > } >> > return temps; >> > } >> > >> > >> > In APPLICATION_JSON mode (commented), the output is as expected. >> However, >> > in TEXT_MODE, I get the following message: >> > >> > No message body writer has been found for class >> > org.apache.geronimo.microprofile.metrics.impl.HistogramImpl, >> > ContentType: text/plain >> > >> > I haven't been able to find a way to correct this, especially since the >> > Counter Metric didn't require a body writer. >> > >> > I would appreciate any help. Thanks! >> > >> > Mike. >> > >> > -- >> > *Code*, *Write*, *Cycle*, *Run*, *Drink*, >> > *Sleep ... Repeat* >> > >> > *InfoQ <https://www.infoq.com/> Java Queue Editor* >> > https://about.me/mpredli <http://about.me/mpredli/> >> > https://twitter.com/mpredli >> > https://redlich.net/ >> > https://javasig.org/ >> > *Laissez Les Bon Temps Rouler* >> > >> > -- *Code*, *Write*, *Cycle*, *Run*, *Drink*, *Sleep ... Repeat* *InfoQ <https://www.infoq.com/> Java Queue Editor* https://about.me/mpredli <http://about.me/mpredli/> https://twitter.com/mpredli https://redlich.net/ https://javasig.org/ *Laissez Les Bon Temps Rouler*
