[incubator-pinot] 01/01: Default PinotAdmin Commands to not use System.exit(...).

[xiangfu]

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

xiangfu pushed a commit to branch fixing_admin_cmd_exit_code
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git

commit ec3e8201157e4dcd44ce154d32a35097c90cd32a
Author: Xiang Fu <fx19880...@gmail.com>
AuthorDate: Fri Apr 12 10:43:06 2019 -0700

    Default PinotAdmin Commands to not use System.exit(...).
---
 .../src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java
 
b/pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java
index f490e2c..df4ab08 100644
--- 
a/pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java
+++ 
b/pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java
@@ -104,7 +104,7 @@ public class PinotAdministrator {
   public static void main(String[] args) {
     PinotAdministrator pinotAdministrator = new PinotAdministrator();
     pinotAdministrator.execute(args);
-    if (!System.getProperties().getProperty("pinot.admin.system.exit", 
"true").equalsIgnoreCase("false")) {
+    if (System.getProperties().getProperty("pinot.admin.system.exit", 
"false").equalsIgnoreCase("true")) {
       System.exit(pinotAdministrator.getStatus() ? 0 : 1);
     }
   }


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