[pinot] branch master updated: fixing npe issue during cleanup pinot-admin (#7291)

[jackie]

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

jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new fca88a2  fixing npe issue during cleanup pinot-admin (#7291)
fca88a2 is described below

commit fca88a20e98edf363e9721bf43e0487e9aafd7b6
Author: Xiang Fu <xiangfu.1...@gmail.com>
AuthorDate: Thu Aug 12 16:49:12 2021 -0700

    fixing npe issue during cleanup pinot-admin (#7291)
---
 .../apache/pinot/tools/admin/command/StartServiceManagerCommand.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartServiceManagerCommand.java
 
b/pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartServiceManagerCommand.java
index bb3f868..82c59c7 100644
--- 
a/pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartServiceManagerCommand.java
+++ 
b/pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartServiceManagerCommand.java
@@ -149,7 +149,9 @@ public class StartServiceManagerCommand extends 
AbstractBaseAdminCommand impleme
 
   @Override
   public void cleanup() {
-    _pinotServiceManager.stopAll();
+    if (_pinotServiceManager != null) {
+      _pinotServiceManager.stopAll();
+    }
   }
 
   @Override

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org