ClickHouse / ClickHouse/adbc_clickhouse

Implement `Statement::set_substrait_plan()` (or decide if we want to support it at all)

Open
#21 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
25
Forks
8
Avg merge
2d 37m
Merged PRs (30d)
4

Description

The ADBC driver API allows directly setting a Substrait query plan (presumably) in lieu of SQL: https://docs.rs/adbc_core/0.21.0/adbc_core/trait.Statement.html#tymethod.set_substrait_plan

It's hard to find information on whether ClickHouse supports Substrait or not.

This page would seem to suggest it does: https://gluten.apache.org/archives/v1.3.0/getting-started/clickhouse-backend/

But I can't find any first-party information in the CH docs. It's almost certainly not supported by the HTTP interface.

Regardless, it seems like it's perfectly fine to return `NotImplemented` from here:

* DataBricks: https://github.com/apache/arrow-adbc/blob/2a6178f1533820ca999fef08e4e57d89ee459175/go/adbc/driver/databricks/statement.go#L221-L225
* Snowflake: https://github.com/apache/arrow-adbc/blob/2a6178f1533820ca999fef08e4e57d89ee459175/go/adbc/driver/snowflake/statement.go#L698-L701

So far, this seems to only be supported by FlightSQL: https://github.com/apache/arrow-adbc/blob/2a6178f1533820ca999fef08e4e57d89ee459175/go/adbc/driver/flightsql/flightsql_statement.go#L70

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.