Store state data in zookeeper

[NikitaUnisikhin]

No description provided.

[Alex-Burmak]

Is the PR in draft on purpose? Or you just forgot to click "Ready for review"?

[NikitaUnisikhin]

Yes, I forgot to click "Ready for review"

By the way, what do you think about making the store-state-zk flag and creating a default path for storing statistics? I don’t like that in this case there will be one path for storing statistics for hosts from different shards, but perhaps this problem should be left on the user’s side. But in some scenarios it may be useful.

It would be cool in ch-tools to know which shard the host belongs to. Do we have such an opportunity? Using this the code can be made much better

[Alex-Burmak]

By the way, what do you think about making the store-state-zk flag and creating a default path for storing statistics? I don’t like that in this case there will be one path for storing statistics for hosts from different shards, but perhaps this problem should be left on the user’s side. But in some scenarios it may be useful.

I have no strong preference between the current implementation and described rework. Choose the one you prefer.

It would be cool in ch-tools to know which shard the host belongs to. Do we have such an opportunity? Using this the code can be made much better

Shard name can be obtained from ClickHouse config, "shard" macro. Another option is to add a setting with shard name to ch-tools config.

[NikitaUnisikhin]

I'm inclined to leave the current implementation for flexibility in storing statistics in zk.
@Alex-Burmak I don't have rights to merge. Merge please if there are no more comments on this PR.