Skip to content

Conversation

Enmk
Copy link
Member

@Enmk Enmk commented Jul 3, 2025

Changelog category (leave one):

Bug Fix (user-visible misbehavior in an official stable release)

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

IcebergS3 supports count optimization, but IcebergS3Cluster does not. As a result, the count() result returned in cluster mode may be a multiple of the number of replicas. (ClickHouse#79844 by @wxybear)

Exclude tests:

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

…luster_wrong_count

fix: ObjectStorageCluster table function return wrong count() after applyTrivialCount optimized
@Enmk
Copy link
Member Author

Enmk commented Jul 3, 2025

It looks like upstream's PR introduces changes that we already have in our branch.

@ianton-ru Please take a look.

@ianton-ru
Copy link

Accidently fixed in #761 / #834.

@Enmk Enmk merged commit 0e032cc into antalya-25.3 Jul 4, 2025
403 of 429 checks passed
Enmk added a commit that referenced this pull request Jul 6, 2025
After merging #878
Bringing tests closer to upstream.
Enmk added a commit that referenced this pull request Jul 7, 2025
…ceberg

Antalya 25.3 : Fix integration tests storage iceber (after #878)
@Enmk Enmk changed the title Antalya 25.3 : Backport of #79844 - ObjectStorageCluster table function return wrong count() after applyTrivialCount optimized Antalya 25.3: Backport of #79844 - ObjectStorageCluster table function return wrong count() after applyTrivialCount optimized Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants