You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ja...@apache.org on 2021/08/12 23:49:21 UTC
[pinot] branch master updated: fixing npe issue during cleanup
pinot-admin (#7291)
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 <xi...@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-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org