Skip to content

PIP-458: Add Async Resource List Filtering API to AuthorizationProvider#25286

Open
mattisonchao wants to merge 3 commits intomasterfrom
pip-455-async-resource-list-filtering
Open

PIP-458: Add Async Resource List Filtering API to AuthorizationProvider#25286
mattisonchao wants to merge 3 commits intomasterfrom
pip-455-async-resource-list-filtering

Conversation

@mattisonchao
Copy link
Member

@mattisonchao mattisonchao commented Mar 4, 2026

Motivation

See pip/pip-458.md for the full proposal.

Modifications

Added PIP-458 proposal document.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

…uthorizationProvider

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Mar 4, 2026
@mattisonchao mattisonchao changed the title PIP-455: Add Async Resource List Filtering API to AuthorizationProvider PIP-458: Add Async Resource List Filtering API to AuthorizationProvider Mar 4, 2026
mattisonchao and others added 2 commits March 5, 2026 01:12
- Remove parent field from FilterContext (resource names already contain hierarchy)
- Remove sort from filter example (caller's responsibility)
- Use thenAccept instead of thenAcceptAsync since filterAsync already offloads from metadata thread

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the PIP label Mar 4, 2026
@codelipenghui
Copy link
Contributor

@mattisonchao the proposal link you provided in the PR description is not available.

@codelipenghui codelipenghui added this to the 4.2.0 milestone Mar 4, 2026
Copy link
Contributor

@codelipenghui codelipenghui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lhotari
Copy link
Member

lhotari commented Mar 9, 2026

@mattisonchao Please start a discussion thread on dev mailing list

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs PIP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants