ZookeeperDataSource might not be initialized well unless the ZooKeeper server is active

[zhousiliang163]

Issue Description

Type: bug report or feature request

Describe what happened (or what feature you want)

在初始化ZookeeperDataSource数据源时，如果zk server不能用或者因网络原因导致client与zk server连接超时等，会造成对znode节点数据的监听失败，如下图：

红色标注的地方，可能会执行报错，导致后面的监听器注册失败。

Describe what you expected to happen

是否可以加个连接事件，等连接成功后，再去执行后面的NodeCacheListener注册

How to reproduce it (as minimally and precisely as possible)

Tell us your environment

Anything else we need to know?

[sentinel-bot]

Hi @zhousiliang163, we detect non-English characters in the issue. This comment is an auto translation from @sentinel-bot to help other users to understand this issue.
We encourage you to describe your issue in English which is more friendly to other users.

Currently only supports the ZookeeperDataSource when the zookeeper server is normal.

Issue Description

Type: bug report or feature request

Describe what happened (or what feature you want)

When the ZookeeperDataSource data source is initialized, if the zk server cannot be used or the connection between the client and the zk server is timed out due to network reasons, the monitoring of the data of the znode node may fail, as shown in the following figure:

Where the red is marked, an error may be reported, causing the subsequent listener registration to fail.

Describe what you expected to happen

Whether it is possible to add a connection event, and wait for the connection to succeed, then perform the subsequent NodeCacheListener registration.

How to reproduce it (as minimally and precisely as possible)

Tell us your environment

Anything else we need to know?

[sczyh30]

Hi, thanks for your suggestion. It's reasonable to deal with circumstances on connection error. Would you like to contribute a PR to improve this?

[zhousiliang163]

OK 发自我的 iPhone

…

在 2019年3月19日，16:20，Eric Zhao ***@***.***> 写道： Hi, thanks for your suggestion. It's reasonable to deal with circumstances on connection error. Would you like to contribute a PR to improve this? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.