obelix74 opened a new pull request, #4115: URL: https://github.com/apache/polaris/pull/4115
## What / Why Part of the 3-PR split of the Iceberg metrics work (agreed with EJ, Dmitri, Anand): - PR0 — #5068 — reporter SPI refactor + write-path relocation. - **PR1 (this PR)** — Table Metrics query REST API. - PR2 — #4756 — JDBC implementation of the query SPI. > **Depends on #5068.** This branch is stacked on the PR0 branch, so until #5068 merges the diff here also shows PR0's changes. Review the commits from `feat(metrics): add Table Metrics Reports query REST API (PR1)` onward. ## Changes - OpenAPI spec + generated module under `extensions/metrics-reports/api` (`polaris-extensions-metrics-reports-api`) — the metrics API ships as an optional, extension-scoped feature. - `MetricsQuerySpi` in `extensions/metrics-reports/spi`; a no-op default (`NoOpMetricsQuery`, `@DefaultBean`) in `extensions/metrics-reports/base` so the read path returns an empty page (HTTP 200) until a durable backend is installed. - Thin HTTP→SPI handler `MetricsReportsService` in `runtime/service` (resolves names→ids, authorizes, delegates to `MetricsQuerySpi`). - Read-path authorization in `polaris-core`: `TABLE_READ_METRICS` privilege + `LIST_TABLE_METRICS` operation, wired through the authorizer/RBAC and mirrored in the Ranger extension. ## Testing - `./gradlew build` and `:polaris-runtime-service:intTest` (Quarkus) pass. -- 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]
