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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
