JeremyXin commented on code in PR #9446: URL: https://github.com/apache/seatunnel/pull/9446#discussion_r2153530169
##########
docs/en/connector-v2/source/Clickhouse.md:
##########
@@ -54,14 +54,28 @@ They can be downloaded via install-plugin.sh or from the
Maven central repositor
| Name | Type | Required | Default |
Description
|
|-------------------|--------|----------|------------------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
| host | String | Yes | - |
`ClickHouse` cluster address, the format is `host:port` , allowing multiple
`hosts` to be specified. Such as `"host1:8123,host2:8123"` .
|
-| database | String | Yes | - | The
`ClickHouse` database.
|
-| sql | String | Yes | - | The query
sql used to search data though Clickhouse server.
|
| username | String | Yes | - |
`ClickHouse` user username.
|
| password | String | Yes | - |
`ClickHouse` user password.
|
+| database | String | NO | - | The
`ClickHouse` database.
|
+| table | String | NO | - | The
`ClickHouse` table. If it is a distributed table, the cluster is obtained based
on the table engine. If it is a local table, build the cluster based on the
input `host`
|
Review Comment:
If the `table_path` parameter is used instead, should the `database`
parameter also be removed? Is it uniformly represented by the `table_path`
parameter?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
