Skip to content

chore: use simpler to install CH client + doc update#9560

Merged
sumerman merged 1 commit into
mainfrom
valeriy/dev-tables-ch-client
Oct 7, 2025
Merged

chore: use simpler to install CH client + doc update#9560
sumerman merged 1 commit into
mainfrom
valeriy/dev-tables-ch-client

Conversation

@sumerman
Copy link
Copy Markdown
Contributor

@sumerman sumerman commented Oct 7, 2025

Important

Switch from clickhouse-client to clickhouse binary for ClickHouse operations and update documentation and scripts accordingly.

  • Development Setup:
    • Updated CONTRIBUTING.md to recommend installing clickhouse binary instead of clickhouse-client.
    • Added installation instructions for clickhouse binary using Homebrew on macOS.
  • Scripts:
    • Modified dev-tables.sh to use clickhouse binary instead of clickhouse-client for executing ClickHouse commands.
    • Updated error messages in dev-tables.sh to reflect the change to clickhouse binary.

This description was created by Ellipsis for 7a49d25. You can customize this summary. It will automatically update as commits are pushed.

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Oct 7, 2025
@sumerman sumerman enabled auto-merge October 7, 2025 08:47
@dosubot dosubot Bot added the auto-merge This PR is set to be merged label Oct 7, 2025
@sumerman sumerman disabled auto-merge October 7, 2025 09:11
@dosubot dosubot Bot removed the auto-merge This PR is set to be merged label Oct 7, 2025
@sumerman sumerman merged commit 6a18e66 into main Oct 7, 2025
25 of 31 checks passed
@sumerman sumerman deleted the valeriy/dev-tables-ch-client branch October 7, 2025 09:11
murdore pushed a commit to juspay/langfuse that referenced this pull request Oct 14, 2025
chore: use simplier to install CH client + doc update.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant