Antalya 25.8: Remote initiator improvements 2#1638
Antalya 25.8: Remote initiator improvements 2#1638ianton-ru wants to merge 2 commits intoantalya-25.8from
Conversation
|
@codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
arthurpassos
left a comment
There was a problem hiding this comment.
To the limits of my knowledge, it looks ok
| if (cluster_name.empty()) | ||
| { | ||
| throw Exception( | ||
| ErrorCodes::LOGICAL_ERROR, |
There was a problem hiding this comment.
I know it was already here, but is LOGICAL_ERROR a good pick? Wouldn't BAD_ARGUMENTS be better? I mean, if the error / empty cluster is a code error, then ok. But if it's a missing argument from the user, then we should change
There was a problem hiding this comment.
When user did not set cluster name, updateQueryForDistributedEngineIfNeeded must be called with make_cluster_function=false, this check is not executed in this case. It is a technical check to be sure I did not make mistake somewhere in code.
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Use
object_storage_remote_initiatorwithoutobject_storage_clusteron initial node.Documentation entry for user-facing changes
Solved #1607
Query with
object_storage_remote_initiatorandobject_storage_remote_initiator_clustercan be executed, whenobject_storage_clustersetting is defined only on remote node.converted to
Remote node
random_node_from_swarm_external_nameexecutesas simple table function or as cluster table function depend on
object_storage_clusterdefined only on this node.CI/CD Options
Exclude tests:
Regression jobs to run: