Skip to content

Antalya 26.4: Support for 'time' type in Iceberg#1892

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

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

Conversation

@zvonand

@zvonand zvonand commented Jun 7, 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 7, 2026 21:48
…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:
#	tests/integration/test_database_iceberg/test.py
The conflict in `tests/integration/test_database_iceberg/test.py` arose
because `antalya-26.4` ends the file with `test_invalid_auth_header_format`
while the source PR's merge commit bundled the tail of `test_cluster_joins`
(not in PR #1761's diff) immediately before the two new test functions.

Kept "ours" (the end of `test_invalid_auth_header_format`) and added the
two new functions from the source PR — `test_partitioning_by_time` and
`test_partitioning_by_string` — after it. Dropped the `test_cluster_joins`
tail from "theirs" as it was noise from the merge commit, not part of
PR #1761's diff.
@zvonand zvonand added releasy Created/managed by RelEasy antalya-26.4 ai-resolved Port conflict auto-resolved by Claude labels Jun 7, 2026
@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown

Workflow [PR], commit [98b62d7]

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.4 releasy Created/managed by RelEasy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant