This is an automated email from the ASF dual-hosted git repository.
youling1128 pushed a commit to branch 2.8.x
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git
The following commit(s) were added to refs/heads/2.8.x by this push:
new 9f056ec32 [#4910] fixed config center response exception cause
nullPointerException (#4912)
9f056ec32 is described below
commit 9f056ec32f27020d13391dd2070cdc63305aa7f3
Author: Alex <[email protected]>
AuthorDate: Mon Sep 1 10:39:21 2025 +0800
[#4910] fixed config center response exception cause nullPointerException
(#4912)
---
.../org/apache/servicecomb/config/center/client/ConfigCenterClient.java | 2 +-
.../apache/servicecomb/config/center/client/ConfigCenterManager.java | 2 +-
.../java/org/apache/servicecomb/config/kie/client/KieConfigManager.java | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git
a/clients/config-center-client/src/main/java/org/apache/servicecomb/config/center/client/ConfigCenterClient.java
b/clients/config-center-client/src/main/java/org/apache/servicecomb/config/center/client/ConfigCenterClient.java
index 6cd4f45a9..7763f5ec6 100644
---
a/clients/config-center-client/src/main/java/org/apache/servicecomb/config/center/client/ConfigCenterClient.java
+++
b/clients/config-center-client/src/main/java/org/apache/servicecomb/config/center/client/ConfigCenterClient.java
@@ -147,7 +147,7 @@ public class ConfigCenterClient implements
ConfigCenterOperation {
} catch (IOException e) {
addressManager.recordFailState(address);
LOGGER.error("query configuration from {} failed, message={}", uri,
e.getMessage());
- throw new OperationException("", e);
+ throw new OperationException("query configuration failed!", e);
}
}
diff --git
a/clients/config-center-client/src/main/java/org/apache/servicecomb/config/center/client/ConfigCenterManager.java
b/clients/config-center-client/src/main/java/org/apache/servicecomb/config/center/client/ConfigCenterManager.java
index 3d60a1468..0ecded522 100644
---
a/clients/config-center-client/src/main/java/org/apache/servicecomb/config/center/client/ConfigCenterManager.java
+++
b/clients/config-center-client/src/main/java/org/apache/servicecomb/config/center/client/ConfigCenterManager.java
@@ -90,7 +90,7 @@ public class ConfigCenterManager extends AbstractTask {
startTask(new
BackOffSleepTask(configCenterConfiguration.getRefreshIntervalInMillis(), new
PollConfigurationTask(0)));
} catch (Exception e) {
LOGGER.warn("get configurations from ConfigCenter failed, and will try
again, cause message: {}. current "
- + "fail does not affect the obtained historical configuration.",
e.getCause().getMessage());
+ + "fail does not affect the obtained historical configuration.",
e.getMessage());
startTask(new BackOffSleepTask(failCount + 1, new
PollConfigurationTask(failCount + 1)));
}
}
diff --git
a/clients/config-kie-client/src/main/java/org/apache/servicecomb/config/kie/client/KieConfigManager.java
b/clients/config-kie-client/src/main/java/org/apache/servicecomb/config/kie/client/KieConfigManager.java
index 72adb86a5..f5a6dfd07 100644
---
a/clients/config-kie-client/src/main/java/org/apache/servicecomb/config/kie/client/KieConfigManager.java
+++
b/clients/config-kie-client/src/main/java/org/apache/servicecomb/config/kie/client/KieConfigManager.java
@@ -156,7 +156,7 @@ public class KieConfigManager extends AbstractTask {
}
} catch (Exception e) {
LOGGER.warn("get configurations from KieConfigCenter failed, and will
try again, cause message: {}. current "
- + "fail does not affect the obtained historical configuration.",
e.getCause().getMessage());
+ + "fail does not affect the obtained historical configuration.",
e.getMessage());
startTask(
new BackOffSleepTask(failCount + 1, new
PollConfigurationTask(failCount + 1, this.configurationsRequest)));
}