Skip to content

Better access validation for TableFunctionLoop#88802

Merged
pufit merged 5 commits intomasterfrom
pufit/fix-table-function-access
Oct 22, 2025
Merged

Better access validation for TableFunctionLoop#88802
pufit merged 5 commits intomasterfrom
pufit/fix-table-function-access

Conversation

@pufit
Copy link
Member

@pufit pufit commented Oct 19, 2025

Changelog category (leave one):

  • Critical Bug Fix (crash, data loss, RBAC) or LOGICAL_ERROR

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Fixes access validation on select for loop table function.

@clickhouse-gh
Copy link
Contributor

clickhouse-gh bot commented Oct 19, 2025 •
edited
Loading

Workflow [PR], commit [285e99f]

Summary: ❌

job_name test_name status info comment
Stateless tests (amd_binary, old analyzer, s3 storage, DatabaseReplicated, parallel) failure
01287_max_execution_speed FAIL cidb, flaky
Integration tests (amd_asan, old analyzer, 5/6) failure
test_zookeeper_config/test_secure.py::test_secure_connection FAIL cidb

@clickhouse-gh clickhouse-gh bot added pr-critical-bugfix pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-must-backport-cloud labels Oct 19, 2025
@evillique evillique self-assigned this Oct 19, 2025
@pufit pufit added this pull request to the merge queue Oct 22, 2025
Merged via the queue into master with commit ae414ae Oct 22, 2025
120 of 123 checks passed
@pufit pufit deleted the pufit/fix-table-function-access branch October 22, 2025 11:46
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 22, 2025
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 mentioned this pull request Oct 22, 2025
Merged
robot-clickhouse-ci-1 added a commit that referenced this pull request Oct 22, 2025
@robot-clickhouse-ci-1
Merge pull request #88885 from ClickHouse/cherrypick/25.3/88802
f3b0e4b 
Cherry pick #88802 to 25.3: Better access validation for `TableFunctionLoop`
robot-clickhouse added a commit that referenced this pull request Oct 22, 2025
@robot-clickhouse
Backport #88802 to 25.3: Better access validation for `TableFunctionL…
0973460 
…oop`
This was referenced Oct 22, 2025
Merged
Merged
robot-clickhouse-ci-1 added a commit that referenced this pull request Oct 22, 2025
@robot-clickhouse-ci-1
Merge pull request #88887 from ClickHouse/cherrypick/25.7/88802
a995a28 
Cherry pick #88802 to 25.7: Better access validation for `TableFunctionLoop`
robot-clickhouse added a commit that referenced this pull request Oct 22, 2025
@robot-clickhouse
Backport #88802 to 25.7: Better access validation for `TableFunctionL…
ecfcaae 
…oop`
This was referenced Oct 22, 2025
Merged
Merged
robot-clickhouse-ci-1 added a commit that referenced this pull request Oct 22, 2025
@robot-clickhouse-ci-1
Merge pull request #88889 from ClickHouse/cherrypick/25.8/88802
6b3789c 
Cherry pick #88802 to 25.8: Better access validation for `TableFunctionLoop`
robot-clickhouse added a commit that referenced this pull request Oct 22, 2025
@robot-clickhouse
Backport #88802 to 25.8: Better access validation for `TableFunctionL…
8edd4d4 
…oop`
This was referenced Oct 22, 2025
Merged
Merged
robot-clickhouse-ci-1 added a commit that referenced this pull request Oct 22, 2025
@robot-clickhouse-ci-1
Merge pull request #88891 from ClickHouse/cherrypick/25.9/88802
cf604d1 
Cherry pick #88802 to 25.9: Better access validation for `TableFunctionLoop`
robot-clickhouse added a commit that referenced this pull request Oct 22, 2025
@robot-clickhouse
Backport #88802 to 25.9: Better access validation for `TableFunctionL…
60fe19c 
…oop`
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 mentioned this pull request Oct 22, 2025
Merged
clickhouse-gh bot added a commit that referenced this pull request Oct 22, 2025
@clickhouse-gh
Merge pull request #88888 from ClickHouse/backport/25.7/88802
8b56808 
Backport #88802 to 25.7: Better access validation for `TableFunctionLoop`
@robot-ch-test-poll2 robot-ch-test-poll2 added pr-backports-created-cloud deprecated label, NOOP pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore labels Oct 22, 2025
pufit added a commit that referenced this pull request Oct 23, 2025
@pufit
Merge pull request #88892 from ClickHouse/backport/25.9/88802
f95a353 
Backport #88802 to 25.9: Better access validation for `TableFunctionLoop`
pufit added a commit that referenced this pull request Oct 23, 2025
@pufit
Merge pull request #88890 from ClickHouse/backport/25.8/88802
b8b9ce2 
Backport #88802 to 25.8: Better access validation for `TableFunctionLoop`
pufit added a commit that referenced this pull request Oct 23, 2025
@pufit
Merge pull request #88886 from ClickHouse/backport/25.3/88802
551a9ed 
Backport #88802 to 25.3: Better access validation for `TableFunctionLoop`
mkmkme pushed a commit to Altinity/ClickHouse that referenced this pull request Oct 23, 2025
@pufit @mkmkme
Merge pull request ClickHouse#88802 from ClickHouse/pufit/fix-table-f…
26b49de 
…unction-access

Better access validation for `TableFunctionLoop`
@mkmkme mkmkme mentioned this pull request Oct 23, 2025
Merged
25 tasks
@zvonand zvonand mentioned this pull request Oct 23, 2025
Merged
zvonand added a commit to Altinity/ClickHouse that referenced this pull request Oct 23, 2025
@zvonand
Merge pull request #1099 from Altinity/backport/24.8.14/88802
5e3b9b5 
24.8.14 Backport of ClickHouse#88802 - Better access validation for `TableFunctionLoop`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@evillique evillique evillique approved these changes

Assignees

@evillique evillique

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-backports-created-cloud deprecated label, NOOP pr-critical-bugfix pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pufit @evillique @robot-ch-test-poll2 @robot-ch-test-poll3