xuesongxs commented on code in PR #4355:
URL: https://github.com/apache/incubator-inlong/pull/4355#discussion_r881289798
##########
inlong-sdk/dataproxy-sdk/src/main/java/org/apache/inlong/sdk/dataproxy/config/ProxyConfigManager.java:
##########
@@ -588,8 +588,12 @@ private ProxyConfigEntry getLocalProxyListFromFile(String
filePath) throws Excep
Map<String, Integer> streamIdMap = getStreamIdMap(localProxyAddrJson);
proxyEntry.setGroupIdNumAndStreamIdNumMap(groupIdNum, streamIdMap);
proxyEntry.setLoad(load);
- if (localProxyAddrJson.has("cluster_id")) {
-
proxyEntry.setClusterId(localProxyAddrJson.get("cluster_id").getAsString());
+ if (localProxyAddrJson.has("data")) {
Review Comment:
> The configuration of the cluster_id needs to be added to the local
configuration file to deal with this problem. Are you trying to deal with the
configuration pulled back from the manager? If this case, you need to modify
the requestProxyList function.
Yes, I changed the wrong place before. Now I only change the
requestproxyList function, but there is another problem, in The .local file is
called cluster_id, in .proxyip and .managerip files it is called clusterId. Is
it necessary to unify here? If so, it should also be modified in the
description of the official website.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
