Skip to content

fix: docker compose build reference to old chdb-rust ref - #109

Merged
austin-barrington merged 1 commit into
mainfrom
fix/docker-compose
Jul 31, 2026
Merged

fix: docker compose build reference to old chdb-rust ref#109
austin-barrington merged 1 commit into
mainfrom
fix/docker-compose

Conversation

@austin-barrington

Copy link
Copy Markdown
Member

Description

Docker compose is cloning and using a hyperbytedb chdb rust lib, our changes were merged into main upstream so we can remove the branch dependencies.

@cursor

cursor Bot commented Jul 31, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Build and lockfile-only changes; risk is mainly that main chdb-rust could diverge from what was tested on the feature branch until CI passes.

Overview
Switches the default chdb-rust git ref from feat_arrow_insert to main everywhere builds clone that path dependency: root Dockerfile, hyperbytedb-proxy/Dockerfile, scripts/checkout-chdb-rust.sh, and .github/scripts/checkout-chdb-rust.sh. CI, local checkout scripts, and docker compose image builds now pull upstream main after the Arrow insert work landed there.

Cargo.lock is refreshed with widespread transitive crate version bumps (HTTP stack, TLS, tokio, serde, etc.)—no application source changes in this diff.

Reviewed by Cursor Bugbot for commit a3e08fd. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Cursor Bugbot completed successfully with no findings requiring human review, and no reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@austin-barrington
austin-barrington merged commit 312a0fb into main Jul 31, 2026
6 checks passed
@austin-barrington
austin-barrington deleted the fix/docker-compose branch July 31, 2026 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant