tiny fix

apache · Dec 11, 2023 · 7be91f2 · 7be91f2
1 parent 7541287
commit 7be91f2
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/checkstyle.xml b/checkstyle.xml
@@ -35,7 +35,8 @@
     <module name="TreeWalker">
         <!--检查行长度-->
         <module name="LineLength">
-            <property name="max" value="105"/>
+            <!-- Use 101 due to some legacy code reason -->
+            <property name="max" value="101"/>
             <!--可以忽略的行-->
             <property name="ignorePattern"
                       value="^package.*|^import.*|a href|href|http://|https://|ftp://"/>

diff --git a/computer-core/src/main/java/org/apache/hugegraph/computer/core/master/MasterService.java b/computer-core/src/main/java/org/apache/hugegraph/computer/core/master/MasterService.java
@@ -156,7 +156,7 @@ public synchronized void close() {
                 this.masterComputation.close(new DefaultMasterContext());
             }
         } catch (Exception e) {
-            LOG.error("Error occurred while closing masterComputation", e);
+            LOG.error("Error occurred while closing master service", e);
         }
 
         if (!failed && this.bsp4Master != null) {

diff --git a/computer-core/src/main/java/org/apache/hugegraph/computer/core/worker/WorkerService.java b/computer-core/src/main/java/org/apache/hugegraph/computer/core/worker/WorkerService.java
@@ -172,9 +172,14 @@ public synchronized void close() {
         }
 
         try {
-            this.computeManager.close();
+            if (this.computeManager != null) {
+                this.computeManager.close();
+            } else {
+                LOG.warn("The computeManager is null");
+                return;
+            }
         } catch (Exception e) {
-            LOG.error("Error when closing computeManager", e);
+            LOG.error("Error when closing ComputeManager", e);
         }
         /*
          * Seems managers.closeAll() would do the following actions: