[GitHub] [incubator-inlong] luchunliang commented on a change in pull request #3138: [INLONG-3136] DataProxy get "NOUPDATE" configuration from Manager when request md5 is same.

[GitBox <gi...@apache.org>]

luchunliang commented on a change in pull request #3138:
URL: https://github.com/apache/incubator-inlong/pull/3138#discussion_r826537841



##########
File path: inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/core/impl/DataProxyClusterServiceImpl.java
##########
@@ -42,15 +44,32 @@
      * @return data proxy config
      */
     public String getAllConfig(String clusterName, String setName, String md5) {
+        log.error("proxyRepository.getClusterSets():{}", new Gson().toJson(proxyRepository.getClusterSets()));
         DataProxyClusterSet setObj = proxyRepository.getDataProxyClusterSet(setName);
+        log.error("proxyRepository.getClusterSets().size():{}, setName:{}, setObj:{}",
+                proxyRepository.getClusterSets().size(), setName, setObj);
         if (setObj == null) {
             return this.getErrorAllConfig();
         }
         String configMd5 = setObj.getMd5Map().get(clusterName);
-        if (configMd5 == null || !configMd5.equals(md5)) {
+        log.error("setObj.getMd5Map().size():{}, clusterName:{}, configMd5:{}", setObj.getMd5Map().size(),

Review comment:
       remove error log




-- 
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: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org