dimas-b commented on code in PR #4115: URL: https://github.com/apache/polaris/pull/4115#discussion_r3477338348
########## 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: Sorry, I missed this before. Let's not change the old default behaviour in a feature PR that just adds new capabilities... This is just of the sake of clarity of intent in this PR. In general, I support changing the default to `no-op`, but let's do it separately. WDYTT? -- 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]
