dimas-b commented on code in PR #4115: URL: https://github.com/apache/polaris/pull/4115#discussion_r3477391313
########## site/content/in-dev/unreleased/telemetry.md: ########## @@ -193,13 +193,19 @@ MDC context is propagated across threads, including in `TaskExecutor` threads. ## Iceberg Metrics Reports API +> **Experimental**: The Metrics Reports REST API and its related Java types (annotated `@Beta`) are +> experimental. They are subject to change, including **breaking changes**, in any future release +> without prior notice. The `@Beta` label indicates early-access / proof-of-concept status, not +> production stability. Do not depend on this API in production environments until it is declared +> stable. + Polaris collects Iceberg scan and commit metrics reports submitted by clients and routes them to a configured reporter. Two built-in reporters are provided: | Type | Description | |------|-------------| -| `log` (default) | Logs each report at INFO level via SLF4J. | -| `no-op` | Silently discards all reports. | +| `log` | Logs each report at INFO level via SLF4J. | +| `no-op` (default) | Silently discards all reports. | Review Comment: Is this misaligned with `runtime/service/src/main/java/org/apache/polaris/service/reporting/MetricsReportingConfiguration.java`? The default in java is `log`, it seems? :thinking: -- 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]
