Hi All, Any remaining blockers on [4115]?
Given that the review and related dev emails discussions [1] were very long, please restate any remaining concerns (here or in GH). Otherwise, I propose merging [4115] on Oct 1. [1] https://lists.apache.org/thread/fhk3rqt7dpdykqs89owmmgpqlv98sg9v [4115] https://github.com/apache/polaris/pull/4115 Thanks, Dmitri. On Fri, Sep 25, 2026 at 3:24 PM Dmitri Bourlatchkov <[email protected]> wrote: > Hi Yufei, > > The Metrics REST API proposal itself was reviewed many months ago [1]. > > However, the proposal did not include Management API changes > initially. This email is to flag only the new grants in the Management API, > which got added recently on EJ's review feedback. > > [1] https://lists.apache.org/thread/fhk3rqt7dpdykqs89owmmgpqlv98sg9v > > Cheers, > Dmitri. > > On Fri, Sep 25, 2026 at 1:47 PM Yufei Gu <[email protected]> wrote: > >> Thanks for raising this, Dmitri. To clarify, *this PR introduces a new >> REST >> spec for metrics consumption*, it is not just adding metrics grants. I >> suggest more people review the spec change, which is critical for any >> users >> who want to consume Iceberg table metrics in Polaris. I will take a look >> soon as well. >> >> Yufei >> >> >> On Fri, Sep 25, 2026 at 7:31 AM Dmitri Bourlatchkov <[email protected]> >> wrote: >> >> > Hi All, >> > >> > Starting a new thread for [1] to increase visibility. This is related >> to PR >> > [4115]. >> > >> > [1] https://lists.apache.org/thread/c5jq95qwzn5dtc103rzk457gr8r0d6zh >> > >> > [4115] https://github.com/apache/polaris/pull/4115 >> > >> > From my POV we're good to merge. >> > >> > Please respond if you have any concerns. >> > >> > Thanks, >> > Dmitri. >> > >> >
