Skip to content

[#9337] improvement: Plan scan API supports access control - #9361

Merged
FANNG1 merged 1 commit into
apache:mainfrom
qqqttt123:ISSUE-9337
Dec 3, 2025
Merged

FANNG1 merged 1 commit into
apache:mainfrom
qqqttt123:ISSUE-9337

Conversation

@roryqi

@roryqi roryqi commented Dec 3, 2025 •

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Plan scan API supports access control

Why are the changes needed?

Fix: #9337

Does this PR introduce any user-facing change?

No need.

How was this patch tested?

Iceberg client don't support remote plan scan now. Now we can't test it yet. You can see apache/iceberg#13400

@roryqi
roryqi requested a review from FANNG1 December 3, 2025 04:02
@FANNG1
FANNG1 merged commit 28ccb5c into apache:main Dec 3, 2025
26 checks passed
@roryqi

roryqi commented Dec 4, 2025

Copy link
Copy Markdown
Contributor Author

I have tested by hand for API. The access control takes effect.

@roryqi
roryqi deleted the ISSUE-9337 branch December 4, 2025 06:27
danhuawang pushed a commit to danhuawang/gravitino that referenced this pull request Jun 8, 2026
…che#9361)

### What changes were proposed in this pull request?

Plan scan API supports access control

### Why are the changes needed?

Fix: apache#9337 

### Does this PR introduce _any_ user-facing change?

No need.

### How was this patch tested?

Iceberg client don't support remote plan scan now. Now we can't test it
yet. You can see apache/iceberg#13400
danhuawang pushed a commit to danhuawang/gravitino that referenced this pull request Jun 9, 2026
…che#9361)

### What changes were proposed in this pull request?

Plan scan API supports access control

### Why are the changes needed?

Fix: apache#9337 

### Does this PR introduce _any_ user-facing change?

No need.

### How was this patch tested?

Iceberg client don't support remote plan scan now. Now we can't test it
yet. You can see apache/iceberg#13400
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Subtask] add authz for scan planning APIs for IRC

2 participants