Looks like we have two threads for the same things. Can we consolidate them?
A question while reviewing PR 5659: How should we authorize metrics queries that span multiple tables, such as all tables under a namespace? A namespace-level TABLE_READ_METRICS grant could cover descendant tables. But if a user only has grants on some tables, should the query return metrics for those tables, or fail because they cannot access the entire namespace? Or is a batch authorization API needed in this case? What behavior do we want the query API to guarantee? Yufei On Wed, Sep 30, 2026 at 2:02 PM Dmitri Bourlatchkov <[email protected]> wrote: > Hi All, > > According to the parallel dev discussion [1] Anand moved Metrics grants > into a separate PR: [5659]. > > Please review. > > [1] https://lists.apache.org/thread/805lmvbvtkt2m57y2gmlnjtcps0ycrz4 > > [5659] https://github.com/apache/polaris/pull/5659 > > Thanks, > Dmitri. > > On Tue, Sep 29, 2026 at 7:52 PM Dmitri Bourlatchkov <[email protected]> > wrote: > > > 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. > >>> > > >>> > >> >
