-
Notifications
You must be signed in to change notification settings - Fork 72
Add chDB Support to MCP ClickHouse Server #51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.
Already on GitHub?
Sign in
to your account
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
f60e02c
to
79a6153
Compare
Hi @iskakaushik , would you mind taking a look at this PR when you have a chance? This PR adds support for chDB alongside the existing ClickHouse functionality, I'd really appreciate your feedback. Thanks! |
@wudidapaopao -- Yup, I will take a look in the next couple of days. @serprex could you give a first look as well please? |
b02b57f
to
271eed1
Compare
3f72965
to
8bb3049
Compare
@iskakaushik @serprex Pls have a look again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wudidapaopao LGTM! Please rebase and we'll merge this!
- Add chDB embedded OLAP engine integration - Implement run_chdb_select_query tool - Add new configurations (CHDB_ENABLED, CHDB_DATA_PATH) - Create comprehensive chDB prompt - Enable hybrid deployments with independent ClickHouse/chDB operation
8bb3049
to
9585e0d
Compare
9585e0d
to
2bb62b7
Compare
@@ -1,5 +1,5 @@ | |||
# Build stage - Use a Python image with uv pre-installed | |||
FROM ghcr.io/astral-sh/uv:python3.13-alpine AS builder | |||
FROM ghcr.io/astral-sh/uv:python3.13-bookworm AS builder |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@serprex This is because chDB does not currently support Alpine Linux as a runtime environment.
This PR introduces comprehensive support for chDB (an embedded SQL OLAP engine) alongside the existing ClickHouse functionality.