Skip to content
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

Feature Request: Support clickhouse-connect's NEW AsyncClient wrapper #324

Open
kdcokenny opened this issue Jul 10, 2024 · 5 comments
Open

Comments

@kdcokenny
Copy link

kdcokenny commented Jul 10, 2024

clickhouse-connect recently released an async wrapper for using the native library asynchronously. I experience many issues with asynch which clickhouse-connect doesn't have. If we could add this wrapper as a dialect that would be great!

https://github.com/ClickHouse/clickhouse-connect/releases/tag/v0.7.16

@xzkostyan
Copy link
Owner

There is async wrapper via asynch: #214

@kdcokenny
Copy link
Author

I understand that but asynch is very buggy and I (and many others it seems) can't use it in production. This is why it would be helpful to have the stable and officially backed clickhouse-connect's new async wrapper supported here.

@xzkostyan
Copy link
Owner

Can you point to specific asynch issues that stop you from using it in production?

@kdcokenny
Copy link
Author

Sure, after about 30 minutes of running asynch with FastAPI it would just stop working and throw the following errors:
asynch.errors.UnexpectedPacketFromServerError: Code: 102. Unexpected packet from server localhost:9000 (expected Pong, got Hello)
and
asynch.errors.ProgrammingError: Code: None. some records have not been fetched. fetch the remaining records before executing the next query

switching to clickhouse's official clickhouse-connect library has worked like a charm.

@ToGoBananas
Copy link

+1 same issue

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

No branches or pull requests

3 participants