ClickHouse / ClickHouse/adbc_clickhouse

Implement `OptionStatement::{Progress, MaxProgress}`

Open
#60 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

I'm not sure how these are actually supposed to work: https://docs.rs/adbc_core/latest/adbc_core/options/enum.OptionStatement.html#variant.Progress

> This represents the progress of execution, not of consumption (i.e., it is independent of how much of the result set has been read by the client).

To me, this would mean getting the progress *while the statement is blocked on execution*, e.g. in `Statement::execute()` or `Statement::execute_update()`, which would be another mutable aliasing issue like with `Statement::cancel()` (#20).

I've asked folks from Columnar for clarification.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the OptionStatement::Progress and MaxProgress documentation, then inspect the Statement::execute() and execute_update() entry points and the cancellation work in issue #20. Resolve the intended execution-progress semantics with the referenced Columnar clarification, including how mutable aliasing is handled. Done means both options have an agreed meaning and an implementation that exposes progress without conflicting with statement execution.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
databases
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.