Skip to content

Follow up to #87303#88017

Merged
novikd merged 1 commit intomasterfrom
row-policy-fix-follow-up
Oct 3, 2025
Merged

Follow up to #87303#88017
novikd merged 1 commit intomasterfrom
row-policy-fix-follow-up

Conversation

@novikd
Copy link
Member

@novikd novikd commented Oct 2, 2025

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

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

Follow up to #87303. Add a check for row policies in ReadFromMergeTree.

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Contributor

clickhouse-gh bot commented Oct 2, 2025

Workflow [PR], commit [d7fe2f9]

Summary:

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Oct 2, 2025
@novikd
Copy link
Member Author

novikd commented Oct 3, 2025

CH Inc sync — tests failed

Unrelated

Copy link
Member

@alexey-milovidov alexey-milovidov left a comment

Choose a reason for hiding this comment

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

Should we add a test?

@alexey-milovidov alexey-milovidov self-assigned this Oct 3, 2025
@novikd novikd added this pull request to the merge queue Oct 3, 2025
Merged via the queue into master with commit 3d380b5 Oct 3, 2025
123 checks passed
@novikd novikd deleted the row-policy-fix-follow-up branch October 3, 2025 15:10
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 3, 2025
mkmkme added a commit to Altinity/ClickHouse that referenced this pull request Oct 15, 2025
This commit is a manual backport of two PRs:
  1. ClickHouse#87303
  2. ClickHouse#88017

Additionally, the test from ClickHouse#88036
was added.
mkmkme added a commit to Altinity/ClickHouse that referenced this pull request Nov 7, 2025
This commit is a manual backport of two PRs:
    1. ClickHouse#87303
    2. ClickHouse#88017

Additionally, the test from ClickHouse#88036
was added.
Also `02679_explain_merge_tree_prewhere_row_policy` was fixed as per
0aed477
zvonand pushed a commit to Altinity/ClickHouse that referenced this pull request Jan 15, 2026
zvonand pushed a commit to Altinity/ClickHouse that referenced this pull request Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants