This is an automated email from the ASF dual-hosted git repository.

healchow pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-inlong.git


The following commit(s) were added to refs/heads/master by this push:
     new af2f854  [INLONG-2993][Manager] Check whether the MQ info is NULL to 
avoid NPE (#2994)
af2f854 is described below

commit af2f85456e974f9ae34134be585aaa311e85dcc5
Author: kuansix <[email protected]>
AuthorDate: Tue Mar 8 15:08:46 2022 +0800

    [INLONG-2993][Manager] Check whether the MQ info is NULL to avoid NPE 
(#2994)
---
 .../inlong/manager/client/api/util/InlongGroupTransfer.java    | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git 
a/inlong-manager/manager-client/src/main/java/org/apache/inlong/manager/client/api/util/InlongGroupTransfer.java
 
b/inlong-manager/manager-client/src/main/java/org/apache/inlong/manager/client/api/util/InlongGroupTransfer.java
index 8e948f5..deb7844 100644
--- 
a/inlong-manager/manager-client/src/main/java/org/apache/inlong/manager/client/api/util/InlongGroupTransfer.java
+++ 
b/inlong-manager/manager-client/src/main/java/org/apache/inlong/manager/client/api/util/InlongGroupTransfer.java
@@ -65,6 +65,9 @@ public class InlongGroupTransfer {
 
     public static MqBaseConf parseMqBaseConf(InlongGroupResponse 
inlongGroupResponse) {
         InlongGroupMqExtBase mqExtBase = inlongGroupResponse.getMqExtInfo();
+        if (null == mqExtBase || 
StringUtils.isBlank(mqExtBase.getMiddlewareType())) {
+            return null;
+        }
         String middleWare = mqExtBase.getMiddlewareType();
         MqType mqType = MqType.forType(middleWare);
         switch (mqType) {
@@ -191,8 +194,11 @@ public class InlongGroupTransfer {
         groupInfo.setMaxLength(groupConf.getMaxLength());
         groupInfo.setProxyClusterId(groupConf.getProxyClusterId());
         MqBaseConf mqConf = groupConf.getMqBaseConf();
-        MqType mqType = mqConf.getType();
-        groupInfo.setMiddlewareType(mqType.name());
+        MqType mqType = MqType.NONE;
+        if (null != mqConf) {
+            mqType = mqConf.getType();
+            groupInfo.setMiddlewareType(mqType.name());
+        }
         groupInfo.setInCharges(groupConf.getOperator());
         groupInfo.setExtList(Lists.newArrayList());
         groupInfo.setCreator(groupConf.getOperator());

Reply via email to