search

ClickHouse / dbt-clickhouse

The Clickhouse plugin for dbt (data build tool)
Apache License 2.0
253 stars 113 forks source link

feat: support range based dictionary layouts #361

Closed rjoelnorgren closed 3 weeks ago

rjoelnorgren commented 1 month ago

The range_hashed and complex_key_range_hashed dictionary layouts require a value is set for RANGE in the ddl. This PR checks the model config for the range key and sets one if provided by the materialization.

BentsiLeviav commented 3 weeks ago

Hi @rjoelnorgren

Thank you for your contribution! Before reviewing your PR, it is required to add a short description with a PR link to the changelog (please keep the current format we have in the Changelog file). If possible, please add a test coverage for this.

rjoelnorgren commented 3 weeks ago

Hi @rjoelnorgren

Thank you for your contribution! Before reviewing your PR, it is required to add a short description with a PR link to the changelog (please keep the current format we have in the Changelog file). If possible, please add a test coverage for this.

@BentsiLeviav, thanks for merging despite being incomplete. Following up here https://github.com/ClickHouse/dbt-clickhouse/pull/376