advancedxy commented on code in PR #684: URL: https://github.com/apache/incubator-uniffle/pull/684#discussion_r1129477032
########## docs/coordinator_guide.md: ########## @@ -130,3 +130,61 @@ PrometheusPushGatewayMetricReporter is one of the built-in metrics reporter, whi |rss.metrics.prometheus.pushgateway.groupingkey|-| Specifies the grouping key which is the group and global labels of all metrics. The label name and value are separated by '=', and labels are separated by ';', e.g., k1=v1;k2=v2. Please ensure that your grouping key meets the [Prometheus requirements](https://prometheus.io/docs/concepts/data_model/#metric-names-and-labels). | |rss.metrics.prometheus.pushgateway.jobname|-| The job name under which metrics will be pushed. | |rss.metrics.prometheus.pushgateway.report.interval.seconds|10| The interval in seconds for the reporter to report metrics. | + +## RESTful API Review Comment: It just hit me that we are going to change a new RESTful web framework, how about add `beta` here Something like `## RESTful API(beta)`. I think we don't commit the RESTful API compatibility here until we settled down a better lightweight and modern RESTful web framework. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
