fix(warehouse): change ClickHouse partition from daily to monthly#6669
fix(warehouse): change ClickHouse partition from daily to monthly#6669deepshekhardas wants to merge 1 commit intorudderlabs:masterfrom
Conversation
…dderlabs#5079) Changed PARTITION BY from toDate() to toYYYYMM() in ClickHouse table creation. This follows ClickHouse best practices recommending partitions of 30-150GB. Monthly partitions are more appropriate for typical rudder-server workloads. Closes rudderlabs#5079
|
Thank you @deepshekhardas for contributing this PR. |
|
Hi @deepshekhardas , thanks for your contribution. |
|
This PR is ready for review. Could a maintainer please take a look? All CI checks are passing. 🙏 |
1 similar comment
|
This PR is ready for review. Could a maintainer please take a look? All CI checks are passing. 🙏 |
|
This PR is ready for review. Could a maintainer please take a look? All CI checks are passing. |
|
This PR is considered to be stale. It has been open 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
2 participants
Closes #5079
Changed PARTITION BY from toDate() to toYYYYMM() in ClickHouse table creation.
This follows ClickHouse best practices recommending partitions of 30-150GB.
Monthly partitions are more appropriate for typical rudder-server workloads.