-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Error in Get Started with Hasura DDN and ClickHouse
Guide
#10669
Comments
Also,
I also tried switching it to the other ports listed by
(However, https://localhost:3280 is accessible and working:) |
Hi @bitjson |
@seanparkross great thanks! Any insight on why |
@bitjson I found an issue that may or may not be related, can you try adding a
Explanation: the clickhouse connector compose file from the tutorial binds data to a directory
I ran into permission issues, because the directory for the database is owned by the database container, but we also try to copy it into the connector container. The Please let me know if the above helps. For me it allows running the |
@BenoitRanque no, still receiving the same error. Trying from scratch: ddn supergraph init my-project && cd my-project
ddn connector init my_ch -i # select ClickHouse
# Connection string: http://local.hasura.dev:8123
# Username: default_user
# Password: default_password
touch app/connector/my_ch/compose.clickhouse.yaml
services:
clickhouse:
image: clickhouse/clickhouse-server
container_name: clickhouse-server
ports:
- "8123:8123"
- "9000:9000"
volumes:
- ./clickhouse-data:/var/lib/clickhouse
environment:
CLICKHOUSE_USER: "default_user"
CLICKHOUSE_PASSWORD: "default_password"
CLICKHOUSE_DB: "default" New step: Added
Verified the Filled DB: curl -u default_user:default_password -X POST \
--data "CREATE TABLE users (user_id UInt32, name String, age UInt8) ENGINE = MergeTree() ORDER BY user_id;" \
http://localhost:8123
curl -u default_user:default_password -X POST \
--data "CREATE TABLE users (user_id UInt32, name String, age UInt8) ENGINE = MergeTree() ORDER BY user_id;" \
http://localhost:8123
curl -u default_user:default_password -X POST \
--data "SELECT * FROM users;" \
http://localhost:8123 Still seeing the error:
Notably, whatever the issue is with I also realized that ![]() You might want to mention excluding |
Seems like a change in ClickHouse 25.1 has caused this https://clickhouse.com/docs/en/whats-new/changelog We're working on a fix. Sorry about the inconvenience! |
@bitjson my bad, able to reproduce after pulling Turns out this is caused by a behavior change in clickhouse 25.1. I've created an issue in their repository While this may be fixed in the future, we'd rather not wait or be dependent on that fix, so we're applying a workaround. |
@bitjson another note: because this is new in clickhouse |
@seanparkross @BenoitRanque ah, good to know, thanks! I'm trying to connect to an existing database that's already on ![]() Is this failure due to the same breaking change in ClickHouse? And general new user question: is there some way to get better visibility into what is happening in the connector? It's logged only one line: |
@bitjson We've just released the fix, please update and let us know if it helps :) That error should only have affected introspection, but the above looks like a runtime error. Can't hurt to try, but I'm expecting that error to persist after the update. Can you confirm that is the case? Have you been able to get local clickhouse to work? My guess is this may be a connection error, perhaps you need to enable an IP?
|
@BenoitRanque yes, the fix works, thanks! And the runtime issue was a network configuration error, now working as expected. Thanks! |
Actually, I think I was still testing against the
|
@bitjson you probably need to upgrade the connector and connector plugin versions, if you have not created a new project.
To shed some light on the issue: hasura uses a plugin architecture for connectors, where each connector provides an executable to use for, amongst other things, introspection. That executable version is set in I can confirm that on my machine, upgrading the version numbers in those files fixes the issue, so it if persists after that we'll have to keep digging |
@BenoitRanque Ah good to know! After ensuring I've updated every other instance of
|
I'm following this tutorial exactly: https://hasura.io/docs/3.0/how-to-build-with-ddn/with-clickhouse
The
ddn connector introspect
command is failing with this error:Versions:
ddn --version
:DDN CLI Version: v2.20.1
(latest)clickhouse/clickhouse-server:latest
issha256:e80057e6e7f35356252afa13a69a42dee47a43652682b67d786ea46611d3a339
In case it's useful:
The text was updated successfully, but these errors were encountered: