Skip to content

Antalya 26.5: Support for 'time' type in Iceberg#1977

Open
zvonand wants to merge 2 commits into
antalya-26.5from
feature/antalya-26.5/pr-1761
Open

Antalya 26.5: Support for 'time' type in Iceberg#1977
zvonand wants to merge 2 commits into
antalya-26.5from
feature/antalya-26.5/pr-1761

Conversation

@zvonand

@zvonand zvonand commented Jun 26, 2026

Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Improvement

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

Support for 'time' type in Iceberg, read and write (#1761 by @ianton-ru).

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)
  • S3 Export (2h)
  • Swarms (30m)
  • Tiered Storage (2h)

Cherry-picked from #1761.


Documentation entry for user-facing changes

Solved #1535

This changes time format,.
Was - seconds from midnight:

SELECT * FROM datalake.`namespace.table`

43200

Now - time with microseconds

SELECT * FROM datalake.`namespace.table`

12:00:00.000000

zvonand added 2 commits June 26, 2026 10:42
…next commit)

---
Original cherry-pick message follows:

Merge pull request #1761 from Altinity/bugfix/antalya-26.3/1535_time_type_write_support

Support for 'time' type in Iceberg
# Conflicts:
#	src/Storages/ObjectStorage/DataLakes/Iceberg/Utils.cpp
#	tests/integration/test_database_iceberg/test.py
In `Utils.cpp`: kept both `DataTypeCustom.h` (antalya-26.5 addition used
for `getCustomName()`) and `DataTypesDecimal.h` (from PR #1761, needed for
`getDecimalScale`).

In `test.py`: kept "ours" end of `test_invalid_auth_header_format` and
discarded the `test_cluster_joins` tail from "theirs" (merge noise not part
of PR #1761); appended `test_partitioning_by_time` and
`test_partitioning_by_string` from PR #1761.
@zvonand zvonand added releasy Created/managed by RelEasy antalya-26.5 ai-resolved Port conflict auto-resolved by Claude labels Jun 26, 2026
@github-actions

Copy link
Copy Markdown

Workflow [PR], commit [c10172a]

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

Labels

ai-resolved Port conflict auto-resolved by Claude antalya-26.5 releasy Created/managed by RelEasy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant