Skip to content

Conversation

@ianton-ru
Copy link

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 to CHANGELOG.md):

Solved #1210
Fix segfault in requests to system,table when table in Iceberg catalog does not have snapshot.

CI/CD Options

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

Regression jobs to run:

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

@github-actions
Copy link

Workflow [PR], commit [4619924]

@ianton-ru
Copy link
Author

Failed test - 03644_object_storage_correlated_subqueries unrelated (fix in #1203)

@zvonand zvonand merged commit 8370e98 into antalya-25.8 Dec 11, 2025
230 of 235 checks passed
@alsugiliazova
Copy link
Member

alsugiliazova commented Dec 17, 2025

Was not able to reproduce this issue, but found that partition_key and sorting_key are not exposed when using glue: #1235

I checked the code, relevant_snapshot variable is used safely now everywhere.

In case of rest, for the reproduction I tried to remove current_snapshot_id field from metadata.json or set it to 1.
In this case SHOW TABLES just returned 0 rows.

In case of glue, SHOW TABLES does not go to object storage, but uses glue metadata. I tried to modify some fields there (Parameters: metadata_location), or just corrupt it, but apparently SHOW TABLES only need table name and namespace to work correctly.

@alsugiliazova alsugiliazova added the verified Verified by QA label Dec 17, 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.

5 participants