daniel-augusto commented on a change in pull request #661:
URL: https://github.com/apache/tomee/pull/661#discussion_r432684881
##########
File path: examples/mp-metrics-histogram/README_pt.adoc
##########
@@ -0,0 +1,452 @@
+= Métricas com MicroProfile `@Histogram`
+:index-group: MicroProfile
+:jbake-type: page
+:jbake-status: published
+
+Este é um exemplo sobre como utilizar as métricas do MicroProfile no TomEE
+Este es un ejemplo sobre cómo utilizar las métricas de MicroProfile en TomEE.
Review comment:
Delete this line.
##########
File path: examples/mp-metrics-histogram/README_pt.adoc
##########
@@ -0,0 +1,452 @@
+= Métricas com MicroProfile `@Histogram`
+:index-group: MicroProfile
+:jbake-type: page
+:jbake-status: published
+
+Este é um exemplo sobre como utilizar as métricas do MicroProfile no TomEE
+Este es un ejemplo sobre cómo utilizar las métricas de MicroProfile en TomEE.
+
+== Executando a aplicação:
+
+[source,bash]
+----
+$ mvn clean install tomee:run
+----
+
+Dentro da aplicação, há um endpoint que te vai da um histograma do clima das
mais recentes temperaturas na cidade de Nova York.
Review comment:
...vai te dar...
##########
File path: examples/mp-metrics-histogram/README_pt.adoc
##########
@@ -0,0 +1,452 @@
+= Métricas com MicroProfile `@Histogram`
+:index-group: MicroProfile
+:jbake-type: page
+:jbake-status: published
+
+Este é um exemplo sobre como utilizar as métricas do MicroProfile no TomEE
+Este es un ejemplo sobre cómo utilizar las métricas de MicroProfile en TomEE.
+
+== Executando a aplicação:
+
+[source,bash]
+----
+$ mvn clean install tomee:run
+----
+
+Dentro da aplicação, há um endpoint que te vai da um histograma do clima das
mais recentes temperaturas na cidade de Nova York.
+
+== Requisição:
+
+[source,bash]
+----
+$ curl -X GET http://localhost:8080/mp-metrics-histogram/weather/histogram
+----
+
+== Resposta:
+
+[source,javascript]
+----
+{
+ "count":15,
+ "max":55,
+ "mean":44.4,
+ "min":27,
+ "p50":45.0,
+ "p75":46.0,
+ "p95":54.0,
+ "p98":54.0,
+ "p99":54.0,
+ "p999":54.0,
+ "stddev":7.0710678118654755,
+ "unit":"degrees F"
+}
+----
+
+== Usando `@Histogram`
+
+As métricas do MicroProfile tem uma função que permite criar um histogram de
dados.
+
+Para utilizar esta função, injetamos um objeto `MetricRegistry`, registrar o
histogram e
+Para utilizar esta función, injecta un objeto `MetricRegistry`, registra el
Histograma, e adiciona dados ao histograma, como mostrado abaixo.
Review comment:
Review this line. My suggestion is:
Para utilizar esta função, injetamos um objeto `MetricRegistry`, registramos
um `Histogram` e adicionamos dados ao histograma como mostrado abaixo.
##########
File path: examples/mp-metrics-histogram/README_pt.adoc
##########
@@ -0,0 +1,452 @@
+= Métricas com MicroProfile `@Histogram`
+:index-group: MicroProfile
+:jbake-type: page
+:jbake-status: published
+
+Este é um exemplo sobre como utilizar as métricas do MicroProfile no TomEE
+Este es un ejemplo sobre cómo utilizar las métricas de MicroProfile en TomEE.
+
+== Executando a aplicação:
+
+[source,bash]
+----
+$ mvn clean install tomee:run
+----
+
+Dentro da aplicação, há um endpoint que te vai da um histograma do clima das
mais recentes temperaturas na cidade de Nova York.
+
+== Requisição:
+
+[source,bash]
+----
+$ curl -X GET http://localhost:8080/mp-metrics-histogram/weather/histogram
+----
+
+== Resposta:
+
+[source,javascript]
+----
+{
+ "count":15,
+ "max":55,
+ "mean":44.4,
+ "min":27,
+ "p50":45.0,
+ "p75":46.0,
+ "p95":54.0,
+ "p98":54.0,
+ "p99":54.0,
+ "p999":54.0,
+ "stddev":7.0710678118654755,
+ "unit":"degrees F"
+}
+----
+
+== Usando `@Histogram`
+
+As métricas do MicroProfile tem uma função que permite criar um histogram de
dados.
Review comment:
Change histogram to histograma
##########
File path: examples/mp-metrics-histogram/README_pt.adoc
##########
@@ -0,0 +1,452 @@
+= Métricas com MicroProfile `@Histogram`
+:index-group: MicroProfile
+:jbake-type: page
+:jbake-status: published
+
+Este é um exemplo sobre como utilizar as métricas do MicroProfile no TomEE
+Este es un ejemplo sobre cómo utilizar las métricas de MicroProfile en TomEE.
+
+== Executando a aplicação:
+
+[source,bash]
+----
+$ mvn clean install tomee:run
+----
+
+Dentro da aplicação, há um endpoint que te vai da um histograma do clima das
mais recentes temperaturas na cidade de Nova York.
+
+== Requisição:
+
+[source,bash]
+----
+$ curl -X GET http://localhost:8080/mp-metrics-histogram/weather/histogram
+----
+
+== Resposta:
+
+[source,javascript]
+----
+{
+ "count":15,
+ "max":55,
+ "mean":44.4,
+ "min":27,
+ "p50":45.0,
+ "p75":46.0,
+ "p95":54.0,
+ "p98":54.0,
+ "p99":54.0,
+ "p999":54.0,
+ "stddev":7.0710678118654755,
+ "unit":"degrees F"
+}
+----
+
+== Usando `@Histogram`
+
+As métricas do MicroProfile tem uma função que permite criar um histogram de
dados.
+
+Para utilizar esta função, injetamos um objeto `MetricRegistry`, registrar o
histogram e
+Para utilizar esta función, injecta un objeto `MetricRegistry`, registra el
Histograma, e adiciona dados ao histograma, como mostrado abaixo.
+
+[source,java]
+----
+@Inject
+private MetricRegistry registry;
+
+@Inject
+@Metric(name = "temperatures", description = "A histogram metrics example.",
+ displayName = "Histogram of Recent New York Temperatures")
+private Histogram histogram;
+
+@Path("/histogram")
+@GET
+@Produces(MediaType.APPLICATION_JSON)
+public Histogram getTemperatures() {
+ Metadata metadata = new Metadata("temperatures", MetricType.HISTOGRAM,
"degrees F");
+ metadata.setDescription("A histogram of recent New York temperatures.");
+ final int[] RECENT_NEW_YORK_TEMPS = { 46, 45, 50, 46, 45, 27, 30, 48, 55,
54, 45, 41, 45, 43, 46 };
+ histogram = registry.histogram(metadata);
+ for(int temp : RECENT_NEW_YORK_TEMPS) {
+ histogram.update(temp);
+ }
+ return histogram;
+}
+----
+
+Existem algumas configurações definidas na anotação `@Histogram`:
+
+*String name* Opcional. Define o nome da métrica. Se não for fornecido
+explicitamente, o nome do objeto anotado é usado.
+
+*String displayName* Opcional. Um nome de exibição legível para metadados.
+
+*String description* Opcional. Uma descrição da métrica.
+
+*String[] tags* Opcional. Matriz de cadeia no formato = para fornecer
+etiquetas especiais para uma métrica.
+
+*boolean reusable* Indica se uma métrica com um determinado nome pode ser
+registre-se em mais de um local. Não se aplica ao `@Histogram`.
+
+=== GET /histogram/status:
+
+[source,bash]
+----
+$ curl -X GET
http://localhost:8080/mp-metrics-histogram/weather/histogram/status
+----
+
+=== Resposta:
+
+[source,bash]
+----
+ Here are the most recent New York City temperatures.
+----
+
+=== Formato Prometheus:
+
+[source,text]
+----
+ # TYPE application:temperatures_degrees F summary histogram
+ # TYPE application:temperatures_degrees F_count histogram
+ application:temperatures_degrees F_count 15.0
+ # TYPE application:temperatures_min_degrees F histogram
+ application:temperatures_min_degrees F 27.0
+ # TYPE application:temperatures_max_degrees F histogram
+ application:temperatures_max_degrees F 55.0
+ # TYPE application:temperatures_mean_degrees F histogram
+ application:temperatures_mean_degrees F 44.4
+ # TYPE application:temperatures_stddev_degrees F histogram
+ application:temperatures_stddev_degrees F 7.0710678118654755
+ # TYPE application:temperatures_degrees F histogram
+ application:temperatures_degrees F{quantile="0.5"} 45.0
+ # TYPE application:temperatures_degrees F histogram
+ application:temperatures_degrees F{quantile="0.75"} 46.0
+ # TYPE application:temperatures_degrees F histogram
+ application:temperatures_degrees F{quantile="0.95"} 54.0
+ # TYPE application:temperatures_degrees F histogram
+ application:temperatures_degrees F{quantile="0.98"} 54.0
+ # TYPE application:temperatures_degrees F histogram
+ application:temperatures_degrees F{quantile="0.99"} 54.0
+ # TYPE application:temperatures_degrees F histogram
+ application:temperatures_degrees F{quantile="0.999"} 54.0
+ # TYPE application:org_superbiz_histogram_weather_service_temperatures
summary histogram
+ # TYPE
application:org_superbiz_histogram_weather_service_temperatures_count histogram
+ application:org_superbiz_histogram_weather_service_temperatures_count 0.0
+ # TYPE application:org_superbiz_histogram_weather_service_temperatures_min
histogram
+ application:org_superbiz_histogram_weather_service_temperatures_min 0.0
+ # TYPE application:org_superbiz_histogram_weather_service_temperatures_max
histogram
+ application:org_superbiz_histogram_weather_service_temperatures_max 0.0
+ # TYPE
application:org_superbiz_histogram_weather_service_temperatures_mean histogram
+ application:org_superbiz_histogram_weather_service_temperatures_mean NaN
+ # TYPE
application:org_superbiz_histogram_weather_service_temperatures_stddev histogram
+ application:org_superbiz_histogram_weather_service_temperatures_stddev 0.0
+ # TYPE application:org_superbiz_histogram_weather_service_temperatures
histogram
+
application:org_superbiz_histogram_weather_service_temperatures{quantile="0.5"}
0.0
+ # TYPE application:org_superbiz_histogram_weather_service_temperatures
histogram
+
application:org_superbiz_histogram_weather_service_temperatures{quantile="0.75"}
0.0
+ # TYPE application:org_superbiz_histogram_weather_service_temperatures
histogram
+
application:org_superbiz_histogram_weather_service_temperatures{quantile="0.95"}
0.0
+ # TYPE application:org_superbiz_histogram_weather_service_temperatures
histogram
+
application:org_superbiz_histogram_weather_service_temperatures{quantile="0.98"}
0.0
+ # TYPE application:org_superbiz_histogram_weather_service_temperatures
histogram
+
application:org_superbiz_histogram_weather_service_temperatures{quantile="0.99"}
0.0
+ # TYPE application:org_superbiz_histogram_weather_service_temperatures
histogram
+
application:org_superbiz_histogram_weather_service_temperatures{quantile="0.999"}
0.0
+----
+
+=== Formato JSON:
+
+[source,bash]
+----
+$ curl -X GET http://localhost:8080/mp-metrics-histogram/metrics/application
+----
+
+=== Resposta JSON:
+
+[source,javascript]
+----
+{
+ "org.superbiz.histogram.WeatherService.temperatures": {
+ "count":0,
+ "max":0,
+ "min":0,
+ "p50":0.0,
+ "p75":0.0,
+ "p95":0.0,
+ "p98":0.0,
+ "p99":0.0,
+ "p999":0.0,
+ "stddev":0.0,
+ "unit":"none"
+ }
+}
+----
+
+== Metadados da métrica:
+
+Uma métrica terá metadados para que você possa aprender mais sobre ela, como
`displayName`,`descrição`, `tags` etc.
Review comment:
Here need translate descrição? Because the words 'displayName' and
'tags' are not translated. I saw this spanish text without translation for this
words.
My suggestion is change to:
Uma métrica terá metadados para fornecer informação sobre ela como
displayName, description, tags, etc.
##########
File path: examples/mp-metrics-histogram/README_pt.adoc
##########
@@ -0,0 +1,452 @@
+= Métricas com MicroProfile `@Histogram`
+:index-group: MicroProfile
+:jbake-type: page
+:jbake-status: published
+
+Este é um exemplo sobre como utilizar as métricas do MicroProfile no TomEE
+Este es un ejemplo sobre cómo utilizar las métricas de MicroProfile en TomEE.
+
+== Executando a aplicação:
+
+[source,bash]
+----
+$ mvn clean install tomee:run
+----
+
+Dentro da aplicação, há um endpoint que te vai da um histograma do clima das
mais recentes temperaturas na cidade de Nova York.
+
+== Requisição:
+
+[source,bash]
+----
+$ curl -X GET http://localhost:8080/mp-metrics-histogram/weather/histogram
+----
+
+== Resposta:
+
+[source,javascript]
+----
+{
+ "count":15,
+ "max":55,
+ "mean":44.4,
+ "min":27,
+ "p50":45.0,
+ "p75":46.0,
+ "p95":54.0,
+ "p98":54.0,
+ "p99":54.0,
+ "p999":54.0,
+ "stddev":7.0710678118654755,
+ "unit":"degrees F"
+}
+----
+
+== Usando `@Histogram`
+
+As métricas do MicroProfile tem uma função que permite criar um histogram de
dados.
+
+Para utilizar esta função, injetamos um objeto `MetricRegistry`, registrar o
histogram e
+Para utilizar esta función, injecta un objeto `MetricRegistry`, registra el
Histograma, e adiciona dados ao histograma, como mostrado abaixo.
+
+[source,java]
+----
+@Inject
+private MetricRegistry registry;
+
+@Inject
+@Metric(name = "temperatures", description = "A histogram metrics example.",
+ displayName = "Histogram of Recent New York Temperatures")
+private Histogram histogram;
+
+@Path("/histogram")
+@GET
+@Produces(MediaType.APPLICATION_JSON)
+public Histogram getTemperatures() {
+ Metadata metadata = new Metadata("temperatures", MetricType.HISTOGRAM,
"degrees F");
+ metadata.setDescription("A histogram of recent New York temperatures.");
+ final int[] RECENT_NEW_YORK_TEMPS = { 46, 45, 50, 46, 45, 27, 30, 48, 55,
54, 45, 41, 45, 43, 46 };
+ histogram = registry.histogram(metadata);
+ for(int temp : RECENT_NEW_YORK_TEMPS) {
+ histogram.update(temp);
+ }
+ return histogram;
+}
+----
+
+Existem algumas configurações definidas na anotação `@Histogram`:
+
+*String name* Opcional. Define o nome da métrica. Se não for fornecido
+explicitamente, o nome do objeto anotado é usado.
+
+*String displayName* Opcional. Um nome de exibição legível para metadados.
+
+*String description* Opcional. Uma descrição da métrica.
+
+*String[] tags* Opcional. Matriz de cadeia no formato = para fornecer
+etiquetas especiais para uma métrica.
+
+*boolean reusable* Indica se uma métrica com um determinado nome pode ser
Review comment:
Change registre-se to registrado. Doubt: This can be used in histogram?
Because of comment "Não se aplica ao `Histogram` ou ao `Gauge`"
##########
File path: examples/mp-metrics-histogram/README_pt.adoc
##########
@@ -0,0 +1,452 @@
+= Métricas com MicroProfile `@Histogram`
+:index-group: MicroProfile
+:jbake-type: page
+:jbake-status: published
+
+Este é um exemplo sobre como utilizar as métricas do MicroProfile no TomEE
+Este es un ejemplo sobre cómo utilizar las métricas de MicroProfile en TomEE.
+
+== Executando a aplicação:
+
+[source,bash]
+----
+$ mvn clean install tomee:run
+----
+
+Dentro da aplicação, há um endpoint que te vai da um histograma do clima das
mais recentes temperaturas na cidade de Nova York.
+
+== Requisição:
+
+[source,bash]
+----
+$ curl -X GET http://localhost:8080/mp-metrics-histogram/weather/histogram
+----
+
+== Resposta:
+
+[source,javascript]
+----
+{
+ "count":15,
+ "max":55,
+ "mean":44.4,
+ "min":27,
+ "p50":45.0,
+ "p75":46.0,
+ "p95":54.0,
+ "p98":54.0,
+ "p99":54.0,
+ "p999":54.0,
+ "stddev":7.0710678118654755,
+ "unit":"degrees F"
+}
+----
+
+== Usando `@Histogram`
+
+As métricas do MicroProfile tem uma função que permite criar um histogram de
dados.
+
+Para utilizar esta função, injetamos um objeto `MetricRegistry`, registrar o
histogram e
+Para utilizar esta función, injecta un objeto `MetricRegistry`, registra el
Histograma, e adiciona dados ao histograma, como mostrado abaixo.
+
+[source,java]
+----
+@Inject
+private MetricRegistry registry;
+
+@Inject
+@Metric(name = "temperatures", description = "A histogram metrics example.",
+ displayName = "Histogram of Recent New York Temperatures")
+private Histogram histogram;
+
+@Path("/histogram")
+@GET
+@Produces(MediaType.APPLICATION_JSON)
+public Histogram getTemperatures() {
+ Metadata metadata = new Metadata("temperatures", MetricType.HISTOGRAM,
"degrees F");
+ metadata.setDescription("A histogram of recent New York temperatures.");
+ final int[] RECENT_NEW_YORK_TEMPS = { 46, 45, 50, 46, 45, 27, 30, 48, 55,
54, 45, 41, 45, 43, 46 };
+ histogram = registry.histogram(metadata);
+ for(int temp : RECENT_NEW_YORK_TEMPS) {
+ histogram.update(temp);
+ }
+ return histogram;
+}
+----
+
+Existem algumas configurações definidas na anotação `@Histogram`:
+
+*String name* Opcional. Define o nome da métrica. Se não for fornecido
+explicitamente, o nome do objeto anotado é usado.
+
+*String displayName* Opcional. Um nome de exibição legível para metadados.
+
+*String description* Opcional. Uma descrição da métrica.
+
+*String[] tags* Opcional. Matriz de cadeia no formato = para fornecer
+etiquetas especiais para uma métrica.
+
+*boolean reusable* Indica se uma métrica com um determinado nome pode ser
+registre-se em mais de um local. Não se aplica ao `@Histogram`.
Review comment:
The original text has: Does not apply to **gauges** or histograms. Here
the citation to the gauges was omitted. The same problem is in Spanish text.
Change to "Não se aplica ao `@Histogram`" to "Não se aplica a gauges ou
histogramas.". Need translate gauge to medidor? I dont know.
And in the original text not refer to object of type `@Histogram` but only
the word histogram. The same problem is in Spanish text.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
