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
Implement SYSTEM DROP REPLICA from auxillary ZooKeeper clusters #48932
Implement SYSTEM DROP REPLICA from auxillary ZooKeeper clusters #48932
Conversation
Can you please add an integration test? |
ok. |
I want to add an integration test, but find that it(tests/integration/helpers/cluster.py) doesn't support creating two sets of zookeeper clusters, Am I right? ClickHouse/tests/integration/helpers/cluster.py Line 2263 in 69a3823
I want to create two separate zookeeper clusters, as shown below, is it impossible to execute?
|
@alexey-milovidov How to solve this problem? |
I see at least some of the integration tests have auxiliary ZooKeeper clusters, let's do something similar to make a test for this PR. |
I looked at some integration tests for auxiliary ZooKeeper clusters, �such as test_reload_auxiliary_zookeepers, test_fetch_partition_from_auxiliary_zookeeper and test_replicated_merge_tree_with_auxiliary_zookeepers, but found that although they used auxiliary ZooKeeper clusters, they were actually the same set of zookeeper clusters, and I looked at the fact that tests/integration/helpers/cluster.py only created one set of zookeeper clusters. Line 16 in 334b052
ClickHouse/tests/integration/helpers/cluster.py Line 2246 in 334b052
This pr requires two separate zookeeper clusters for testing. |
|
It does not, it's okay to do it the same way as in existing tests. See also |
ok, I add an integration test, please take a look. |
…_fix' into multiple_zookeeper_drop_replica_bug_fix
This is an automated comment for commit 7654771 with description of existing statuses. It's updated for the latest CI running
|
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Implement SYSTEM DROP REPLICA from auxillary ZooKeeper clusters, may be close #48931.