Skip to content

feat: Implement custom ORDER BY clause support.#19674

Merged
kodiakhq[bot] merged 2 commits intomainfrom
mariano/custom-order-by
Nov 21, 2024
Merged

feat: Implement custom ORDER BY clause support.#19674
kodiakhq[bot] merged 2 commits intomainfrom
mariano/custom-order-by

Conversation

@marianogappa
Copy link
Copy Markdown
Contributor

This PR implements support for setting custom ORDER BY clauses on any table or group of tables in the ClickHouse destination.

Because tables are defined on the source side and every destination can have different optimal indexing/sorting circumstances, it is not practical to define destination-specific sorting strategies at the source. Therefore, this enables setting it at the destination.

Screenshot 2024-11-19 at 16 08 06 Screenshot 2024-11-19 at 16 08 32

@marianogappa marianogappa marked this pull request as ready for review November 21, 2024 11:58
@marianogappa marianogappa requested review from a team, erezrokah and maaarcelino and removed request for a team and maaarcelino November 21, 2024 11:58
@cq-bot
Copy link
Copy Markdown
Contributor

cq-bot commented Nov 21, 2024

@marianogappa marianogappa added automerge Automatically merge once required checks pass and removed automerge Automatically merge once required checks pass labels Nov 21, 2024
@marianogappa marianogappa added the automerge Automatically merge once required checks pass label Nov 21, 2024
@kodiakhq kodiakhq bot merged commit 14f8a41 into main Nov 21, 2024
@kodiakhq kodiakhq bot deleted the mariano/custom-order-by branch November 21, 2024 12:12
kodiakhq bot pushed a commit that referenced this pull request Nov 29, 2024
🤖 I have created a release *beep* *boop*
---


## [6.0.0](plugins-destination-clickhouse-v5.2.0...plugins-destination-clickhouse-v6.0.0) (2024-11-29)


### ⚠ BREAKING CHANGES

* Fix the order by bug that causes compound types to always be included in the sort key
* Detect changes to order and partition by and fail migration (or recreate tables in `force_migrate` mode)

### Features

* Detect changes to order and partition by and fail migration (or recreate tables in `force_migrate` mode) ([8c355f7](8c355f7))
* Implement custom ORDER BY clause support. ([#19674](#19674)) ([14f8a41](14f8a41))


### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.69.0 ([#19677](#19677)) ([84cd7bd](84cd7bd))
* Fix the order by bug that causes compound types to always be included in the sort key ([8c355f7](8c355f7))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/plugin/destination/clickhouse automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants