Antalya 26.6 - Backport of #115603 - read row_lineage(iceberg v3) fields as virtual columns - #2308
Antalya 26.6 - Backport of #115603 - read row_lineage(iceberg v3) fields as virtual columns#2308subkanthi wants to merge 8 commits into
Conversation
… and DBMS_CLUSTER_PROCESSING_PROTOCOL_VERSION
CI triage for #2308 (Iceberg v3
|
|
TESTING CH After inserting 150 rows, ice does Append |
ianton-ru
left a comment
There was a problem hiding this comment.
Code the same as in original PR, but I recommend also to keep the same format (indents, empty lines, brackets) to avoid additional conflicts in future. I did not mark all places with different formatting.
| } | ||
| } | ||
|
|
||
| std::optional<UInt64> first_row_id; |
There was a problem hiding this comment.
This block a little bit reformatted (removed empty lines, brackets). This does not change something, but can create additional conflicts in future backports instead of automatic solving during git merge/git cherry-pick.
|
|
||
| using namespace DB::Iceberg; | ||
|
|
||
| std::optional<DB::Range> getMaterializedRowLineageRange(const ParsedManifestFileEntry & parsed_entry, Int32 field_id) |
There was a problem hiding this comment.
The same as above - block is reformatted (different indent).
|
|
||
| namespace | ||
| { | ||
| DataTypePtr rowLineageColumnType() |
|
Triage of CI on Status snapshot
Nothing is in a FAILED/ERROR state — I checked the full praktika report ( What still needs to run — the jobs that actually exercise this backport haven't executed yet on this commit:
Note: an earlier run on Health check This is a well-scoped backport of upstream ClickHouse#115603 (read Next step: let the run finish, then re-run — automated CI triage by @blau-ai |
CI triage for #2308 (backport of ClickHouse#115603 — Iceberg v3
|
| Check | What actually happened | Classification |
|---|---|---|
| Stateless — cas s3 / cas / db disk distributed plan / s3 storage (asan, debug, binary) | 1 dataset test per suite timed out on connect timed out: 65.108.242.32:6000 |
Infra (network) |
| Stress test (amd_asan_ubsan) | datasets.visits_v1 load timeout → no test_results.tsv → exit 210; fatal.log empty |
Infra (network) |
| Stateless (amd_debug, sequential) | Only named failure 04051_web_disk_stale_connections; the suite's own diagnosis re-ran it 10×, all passed ("failure is not reproducible") |
Flaky |
| Integration (amd_asan_ubsan, db disk, old analyzer, 6/8) | xdist.dsession.Interrupted: session-timeout: 10800.0 sec exceeded at 92%, tests passing (no Iceberg test involved) |
Infra (3h timeout) |
Regression: cas_selects, cas_lightweight_delete_4, settings, swarms |
Small failure counts (e.g. settings 4/1713, swarms 1/1535) in Altinity CAS/settings/swarms suites — code paths this PR does not modify, run during the same degraded window | Pre‑existing / flaky (not PR‑related) |
PR (aggregate) |
Rolls up the above | n/a |
Recommendation
- No code change is warranted — nothing here points at the backport. Please just re‑run the failed jobs; they should clear once the
65.108.242.32:6000dataset host is reachable and the integration shard isn't starved for time. - One caveat worth flagging: because the integration shards were mostly skipped/timed‑out, this PR's own new Iceberg tests (
test_storage_iceberg_with_spark/test_row_lineage*.py, stateless04203_system_iceberg_files) did not get a clean validation run. A re‑run is needed to actually confirm the backport's new functionality passes — the current red is not evidence for or against it.
Evidence: result_pr.json, clickhouse-server.err.log and job.log for the stateless/stress suites at 868dfb9. I can't build or run ClickHouse in this environment, so this triage is from the CI reports only.
ianton-ru
left a comment
There was a problem hiding this comment.
Can't be merged because of incompatible protocol, see discussion in private slack
https://altinity.slack.com/archives/C089554C9KP/p1789112837538069
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Iceberg v3 has two new fields in manifest file -
first_row_idandlast_seq_num. This PR allows to read them as virtual columns from system tables(system.iceberg_files) (ClickHouse#115603 by @scanhex12)CI/CD Options
Exclude tests:
Regression jobs to run: