-
Notifications
You must be signed in to change notification settings - Fork 4
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
Convert functions with object_storage_cluster setting to cluster functions #712
Conversation
This is an automated comment for commit e61e67a with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page
Successful checks
|
e61e67a
to
480ca5a
Compare
|
||
auto cluster_name = getClusterName(context); | ||
auto cluster_name_arg = std::make_shared<ASTLiteral>(cluster_name); | ||
args.insert(args.begin(), cluster_name_arg); |
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.
What if the user does something creepy like:
select * from s3Cluster("cluster1"...) SETTINGS object_storage_cluster="cluster2";
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.
The diff seems reasonable, just replaces existing function with a clustered version. I don't have the time to check whether this is the right place to apply these changes and etc.
I'll trust your instincts.
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Convert functions with object_storage_cluster setting to cluster functions
Documentation entry for user-facing changes
This allows to use old clickhouse version in swarm cluster. All logic with
object_storage_cluster
setting executed only on initiator, other nodes can work without it.