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

Support load balancing connection string #63

Open
saimon46 opened this issue Nov 2, 2023 · 0 comments
Open

Support load balancing connection string #63

saimon46 opened this issue Nov 2, 2023 · 0 comments

Comments

@saimon46
Copy link

saimon46 commented Nov 2, 2023

Hello,
The JDBC clickhouse driver that the connector uses supports already load balancing, specifing in the connection string multiple hosts and ports.

String connString = "jdbc:clickhouse://server1:8123,server2:8123,server3:8123/database";
BalancedClickhouseDataSource balancedDs = new BalancedClickhouseDataSource(
    connString).scheduleActualization(5000, TimeUnit.MILLISECONDS);
ClickHouseConnection conn = balancedDs.getConnection("default", "");

Is it possible support this driver feature on the connector itself? Now only one server is configurable and the connection string is hardcoded https://github.com/ClickHouse/clickhouse-tableau-connector-jdbc/blob/main/clickhouse_jdbc/connectionBuilder.js#L72

Thanks a lot
Simone

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

1 participant