Skip to content

[codex] Support timezone-qualified RowBinary datetimes - #359

Closed
ruslandoga wants to merge 7 commits into
masterfrom
ruslandoga-conductor/fix-plausible-ch-353
Closed

[codex] Support timezone-qualified RowBinary datetimes#359
ruslandoga wants to merge 7 commits into
masterfrom
ruslandoga-conductor/fix-plausible-ch-353

Conversation

@ruslandoga

@ruslandoga ruslandoga commented May 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Preserve timezone-qualified DateTime and DateTime64 RowBinary encoding types instead of rejecting non-UTC annotations.
  • Interpret NaiveDateTime values in the ClickHouse type timezone while continuing to encode DateTime values by instant.
  • Add focused regression coverage and an Unreleased changelog entry.

Fixes #353.

Checks

  • MIX_ENV=test mix format --check-formatted
  • mix test test/ch/row_binary_test.exs
  • Full suite delegated to isolated CI because concurrent local workspaces share the ClickHouse test database.

@ruslandoga
ruslandoga force-pushed the ruslandoga-conductor/fix-plausible-ch-353 branch from 9e0808c to 0cd6fe0 Compare August 3, 2026 13:03
@ruslandoga
ruslandoga force-pushed the ruslandoga-conductor/fix-plausible-ch-353 branch from 0cd6fe0 to 650ab96 Compare August 9, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support RowBinary encoding for non-UTC DateTime timezones

1 participant