Hi All, +1 to enable LIST_PAGINATION_ENABLED by default
Cheers, Dmitri. On Wed, Sep 16, 2026 at 11:28 PM Yufei Gu <[email protected]> wrote: > Hi everyone, > > Following up on the discussion in PR #5385 > <https://github.com/apache/polaris/pull/5385#discussion_r4021046514>, I’d > like to propose enabling list pagination by default. > > Currently, LIST_PAGINATION_ENABLED defaults to false, so users need to > enable it before clients can use pagination, including the Python CLI’s new > --page-size option. > > Enabling the flag does not force pagination on existing clients: requests > that omit both pageToken and pageSize still receive the full list. Changing > the default would let clients request pagination without an extra server > configuration step, while preserving the behavior of requests that do not > ask for it. > > I see two options: > > 1. > > Default the flag to true, retaining the ability to disable pagination. > 2. > > Remove the flag and always support pagination when requested. > > I lean toward enabling it by default first, keeping the flag as an escape > hatch. We could consider removing it later if there is no remaining need to > disable pagination. > > Are there compatibility concerns, known impl. limitations, or operational > reasons to keep pagination disabled by default? Would others prefer > removing the flag altogether? > Yufei >
