check child is json or not in zookeeper.it will be continue if not.
What is the purpose of the change
catch the exception when the url can't be deserialization.related issue
Brief changelog
add the json check when ANY_VALUE.equals(url.getServiceInterface()) at ZookeeperRegistry#doSubscribe.
Verifying this change
the child will work normal if the child is json.It can still work when dubbo-admin run.
Checklist
- [x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
- [x] Each commit in the pull request should have a meaningful subject line and body.
- [x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
- [x] GitHub Actions works fine on your own branch.
@CrazyHZM hello,In the last commit, I only make the logger for warnings and did not change any logic. However, it still did not pass the checks.That is strange.I checked the logs, and it seems to be a timeout issue with Zookeeper. Could the success of the checks be related to network factors?
@AlbumenJ I catch the exception,but i can't pass the CI/CD.I am trying to fix it.
@AlbumenJ Take a look,please.
Also, pls add some test cases
@AlbumenJ I had finish it,could you take a look please?
Quality Gate passed
Issues
29 New issues
0 Accepted issues
Measures
0 Security Hotspots
67.6% Coverage on New Code
0.0% Duplication on New Code