Skip to content
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

findReplicaHavingCoveringPart may create a lot of excessive traffic #21338

Open
filimonov opened this issue Mar 1, 2021 · 2 comments
Open

findReplicaHavingCoveringPart may create a lot of excessive traffic #21338

filimonov opened this issue Mar 1, 2021 · 2 comments
Labels
comp-zookeeper Zookeeper st-discussion The story requires discussion /research / expert help / design & decomposition before will be taken unexpected behaviour

Comments

@filimonov
Copy link
Contributor

Here we do full parts listing:

Strings parts = zookeeper->getChildren(zookeeper_path + "/replicas/" + replica + "/parts");

If number of parts & replicas is big that creates significant amount of traffic between zookeeper and clickhouse.

I guess we can start with more 'point' lookup by exact part name cluster-wide.

@alex-zaitsev
Copy link
Contributor

alex-zaitsev commented Jun 18, 2021

select event, formatReadableSize(value) 
 from system.events where event in ('InsertedBytes','ZooKeeperBytesSent','ZooKeeperBytesReceived');
┌─event──────────────────┬─formatReadableSize(value)─┐
│ InsertedBytes          │ 89.17 GiB                 │
│ ZooKeeperBytesSent     │ 111.04 GiB                │
│ ZooKeeperBytesReceived │ 13.14 TiB                 │
└────────────────────────┴───────────────────────────┘

@alexey-milovidov alexey-milovidov added the st-discussion The story requires discussion /research / expert help / design & decomposition before will be taken label Jun 18, 2021
filimonov added a commit that referenced this issue Jun 18, 2021
@alex-zaitsev
Copy link
Contributor

Up. Latest results. 140TB of ZK traffic on 500GB of inserts.

┌─event──────────────────┬─formatReadableSize(value)─┐
│ InsertedBytes          │ 515.58 GiB                │
│ ZooKeeperBytesSent     │ 542.53 GiB                │
│ ZooKeeperBytesReceived │ 142.56 TiB                │
└────────────────────────┴───────────────────────────┘

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-zookeeper Zookeeper st-discussion The story requires discussion /research / expert help / design & decomposition before will be taken unexpected behaviour
Projects
None yet
Development

No branches or pull requests

3 participants