-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Bump clickhouse client to 0.9.4 #69829
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
base: master
Are you sure you want to change the base?
Conversation
This should fix issues when loading many tables at the same time, see ClickHouse/clickhouse-java#1741
👋 Welcome to Airbyte!Thank you for your contribution from bouk/airbyte! We're excited to have you in the Airbyte community. Helpful Resources
PR Slash CommandsAs needed or by request, Airbyte Maintainers can execute the following slash commands on your PR:
If you have any questions, feel free to ask in the PR comments or join our Slack community. Tips for Working with CI
|
|
/run-connector-tests
|
|
@bouk thank you for your contribution! |
|
done! |
This should fix issues when loading many tables at the same time. I've not tested this, since I don't have airbyte running locally, just using it
What
I kept getting this issue:
When loading 15 schemas at the same time, I could see from the logs it was doing 20 queries at once:
and this seems to match this clickhouse-java issue I found: ClickHouse/clickhouse-java#1741
How
Upgrade clickhouse client to include the latest fixes
Can this PR be safely reverted and rolled back?