Skip to content

fix: clickhouse quarter partition type#6965

Open
achettyiitr wants to merge 1 commit into
release/1.75.xfrom
fix.clickhouse-quarter-partition-type
Open

fix: clickhouse quarter partition type#6965
achettyiitr wants to merge 1 commit into
release/1.75.xfrom
fix.clickhouse-quarter-partition-type

Conversation

@achettyiitr
Copy link
Copy Markdown
Member

Description

Add support for quarterly partitioning in ClickHouse destinations using `toStartOfQuarter(received_at)` as the partition expression.

Changes

  • `clickhouse.go`: Added `case "quarter"` to `partitionExpr()` returning `toStartOfQuarter(received_at)`
  • `partition_test.go`: Added unit test case for `"quarter"` partition type
  • `clickhouse_test.go`: Added integration test case verifying `toStartOfQuarter(received_at)` partition key in `system.tables`

Linear Ticket

  • Resolves INT-6377

Security

  • The code changed/added as part of this pull request won't create any security issues with how the software is being used.

@achettyiitr achettyiitr changed the base branch from master to release/1.75.x May 18, 2026 10:27
@achettyiitr achettyiitr changed the title Fix.clickhouse quarter partition type fix: clickhouse quarter partition type May 18, 2026
Add support for quarterly partitioning in ClickHouse destinations using
toStartOfQuarter(received_at) as the partition expression.

Resolves INT-6377
@achettyiitr achettyiitr force-pushed the fix.clickhouse-quarter-partition-type branch from e8936c1 to 0ccd0b4 Compare May 18, 2026 10:28
@codecov
Copy link
Copy Markdown

codecov Bot commented May 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.06%. Comparing base (19122c8) to head (0ccd0b4).

Additional details and impacted files
@@                Coverage Diff                 @@
##           release/1.75.x    #6965      +/-   ##
==================================================
+ Coverage           79.68%   80.06%   +0.37%     
==================================================
  Files                 565      565              
  Lines               63672    63674       +2     
==================================================
+ Hits                50740    50983     +243     
+ Misses               9931     9679     -252     
- Partials             3001     3012      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants