Skip to content

Stable 25.8: Backports of #88490 and #89999 - #2094

Merged
strtgbb merged 2 commits into
stable-25.8from
backport/stable-25.8/89999
Jul 22, 2026
Merged

Stable 25.8: Backports of #88490 and #89999#2094
strtgbb merged 2 commits into
stable-25.8from
backport/stable-25.8/89999

Conversation

@CarlosFelipeOR

Copy link
Copy Markdown
Collaborator

Backporting to stable-25.8 because this crash is currently failing the Stress test on our CI (server aborts on transaction commit over non-append disks).

Changelog category (leave one):

  • CI Fix or Improvement (changelog entry is not required)

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

For MergeTree tables using object storage which do not support writing with append, it will not support transactions. (ClickHouse#88490 by @tuanpach)
Check if encrypted disks support writing with append (ClickHouse#89999 by @tuanpach)
Closes ClickHouse#84669

Documentation entry for user-facing changes

...

CI/CD Options

Exclude tests:

  • Fast test
  • Integration Tests
  • Stateless tests
  • Stateful tests
  • Performance tests
  • Aarch64 tests
  • All with ASAN
  • All with TSAN
  • All with MSAN
  • All with UBSAN
  • All with Coverage
  • All Regression
  • Disable CI Cache

Regression jobs to run:

  • Fast suites (mostly <1h)
  • Aggregate Functions (2h)
  • Alter (1.5h)
  • Benchmark (30m)
  • ClickHouse Keeper (1h)
  • Iceberg (2h)
  • LDAP (1h)
  • OAuth (5m)
  • Parquet (1.5h)
  • RBAC (1.5h)
  • SSL Server (1h)
  • S3 (2h)
  • S3 Export (2h)
  • Swarms (30m)
  • Tiered Storage (2h)

…tion-for-non-append-supporting-disks

Not support transaction for tables with disks which do not support writing with append.
@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown

Workflow [PR], commit [f4addda]

ianton-ru
ianton-ru previously approved these changes Jul 21, 2026

@ianton-ru ianton-ru left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The same as #1475 except strange includes in src/Storages/MergeTree/MergeTreeDeduplicationLog.cpp

Comment thread src/Storages/MergeTree/MergeTreeDeduplicationLog.cpp Outdated
Check if encrypted disks support writing with append

@ianton-ru ianton-ru left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@CarlosFelipeOR

Copy link
Copy Markdown
Collaborator Author

CI is clean. The only failing test is 04299_constraint_subst_correlated_subquery_root, due to the known issue #2051.
All stress tests passed, which was the purpose of this PR.
Adding the verified label.

@CarlosFelipeOR CarlosFelipeOR added the verified Approved for release label Jul 21, 2026
@strtgbb
strtgbb merged commit e520847 into stable-25.8 Jul 22, 2026
293 of 296 checks passed
@mkmkme

mkmkme commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

No need to port this to Antalya 25.8, it has already been handled by #1475

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants