Cannot invoke "java.util.Collection.size()" because "storageNodes" is null

[a601942905git]

Question

The following error occurs when shardingsphere integrates consul：Cannot invoke "java.util.Collection.size()" because "storageNodes" is null
config content：

mode:
  type: Cluster
  repository:
    type: consul
    props:
      namespace: governance_ds
      server-lists: localhost
      timeToLiveSeconds: 30
      blockQueryTimeToSeconds: 60

shardingsphere-jdbc-core version is：5.4.1
shardingsphere-cluster-mode-repository-consul version is：5.4.1

[RaigorJiang]

Hi @a601942905git
Do you have configuration in consul, and how to reproduce this exception?

[a601942905git]

What configuration do I need to add on consul

[a601942905git]

Hi @a601942905git Do you have configuration in consul, and how to reproduce this exception?

Consul does not have any configuration. It can be started normally if it is replaced with zookeeper without adding configuration.

[a601942905git]

@RaigorJiang you can see this method：org.apache.shardingsphere.mode.spi.PersistRepository#getChildrenKeys
zookeeper impl return empty collection
consul impl return null

[RaigorJiang]

@RaigorJiang you can see this method：org.apache.shardingsphere.mode.spi.PersistRepository#getChildrenKeys zookeeper impl return empty collection consul impl return null

Hi @a601942905git , do you mean response.getValue() will be null?

[a601942905git]

@RaigorJiang you can see this method：org.apache.shardingsphere.mode.spi.PersistRepository#getChildrenKeys zookeeper impl return empty collection consul impl return null

Hi @a601942905git , do you mean response.getValue() will be null?

yes

[RaigorJiang]

Hi @a601942905git , do you mean response.getValue() will be null?
yes

In this case, we can judge response.getValue() first. Are you interested in submitting a PR?

[a601942905git]

Hi @a601942905git , do you mean response.getValue() will be null?
yes

In this case, we can judge response.getValue() first. Are you interested in submitting a PR?

no

[linghengqian]

• I also addressed the current issue in Allow ConsulRawClient of cluster-mode-repository-consul to be configured on ports other than 8500 #29621.
• However, the current issue does not involve the problem that ConsulRepository.watch() cannot update the local data source. If you are interested, you are welcome to open a new PR or a new issue. See Add GraalVM Reachability Metadata and corresponding nativeTest for Consul integration #29588 .

[linghengqian]

• Call the original implementer of the Consul integration @gavenpeng . Do you have time to deal with issues related to ConsulRepository.watch() failure?