-
Notifications
You must be signed in to change notification settings - Fork 14
Documentation for Swarm features in Antalya branch #1165
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
Conversation
|
@codex review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <named_collections> | ||
| <iceberg_conf> | ||
| <url>http://test.s3.amazonaws.com/clickhouse-bucket/</url> | ||
| <access_key_id>test<access_key_id> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct malformed XML tag in named collection example
The Altinity example for icebergCluster named collections shows <access_key_id>test<access_key_id> without a closing tag, so anyone copying it would end up with invalid XML and ClickHouse will refuse to parse the named collection. Close the tag as </access_key_id> to make the sample usable.
Useful? React with 👍 / 👎.
|
|
||
| icebergAzure(connection_string|storage_account_url, container_name, blobpath, [,account_name], [,account_key] [,format] [,compression_method]) SETTINGS object_storage_cluster='cluster_name' | ||
|
|
||
| icebergHDSF(path_to_table, [,format] [,compression_method]) SETTINGS object_storage_cluster='cluster_name' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
|
||
| icebergLocal(path_to_table, [,format] [,compression_method]) SETTINGS object_storage_cluster='cluster_name' | ||
|
|
||
| icebergS3(option=value [,..]]) SETTINGS object_storage_cluster='cluster_name' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove extra bracket in
icebergS3 settings example
The alternative syntax shows icebergS3(option=value [,..]]) SETTINGS ... with an extra closing bracket; as written this is not valid table function syntax and will raise a parse error if copied. Dropping the stray ] makes the example executable.
Useful? React with 👍 / 👎.
Documentation for Swarm features in Antalya branch
|
QA verified. The cluster_name is missing in one of the examples. A comment has been left. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Documentation for Swarm features in Antalya branch
Documentation entry for user-facing changes
...
CI/CD Options
Exclude tests:
Regression jobs to run: