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

Update ClickHouse driver to support HTTP #16422

Open
AndreKR opened this issue Jan 22, 2025 · 0 comments · May be fixed by #16423
Open

Update ClickHouse driver to support HTTP #16422

AndreKR opened this issue Jan 22, 2025 · 0 comments · May be fixed by #16423
Labels
feature request Requests for new plugin and for new features to existing plugins

Comments

@AndreKR
Copy link

AndreKR commented Jan 22, 2025

Use Case

ClickHouse offers both a "native" and an HTTP API.

The SQL output plugin has a setting driver="clickhouse" that is supposed to be able to push metrics to a ClickHouse database. The database credentials are given using the data_source_name setting.

Expected behavior

If I specify http://... as the DSN I'd expect it to connect to ClickHouse via HTTP and start pushing metrics.

Actual behavior

The SQL output plugin uses an old version of the ClickHouse driver which doesn't support HTTP yet.

It also doesn't actually detect that, so it will happily accept a http://hostname:1234 DSN and connect and try talking native protocol to the HTTP server, causing the whole Telegraf process to hang.

Additional info

No response

@AndreKR AndreKR added the feature request Requests for new plugin and for new features to existing plugins label Jan 22, 2025
@AndreKR AndreKR linked a pull request Jan 22, 2025 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Requests for new plugin and for new features to existing plugins
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant