You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2016/02/10 19:31:09 UTC
[1/4] hbase git commit: HBASE-15229 Canary Tools should not call
System.Exit on error (Vishal Khandelwal)
Repository: hbase
Updated Branches:
refs/heads/0.98 62ce0e21c -> 6bc355721
refs/heads/branch-1 5fe081eb3 -> e85575f56
refs/heads/branch-1.2 8a2cb1608 -> 3b6c3057d
refs/heads/master d53318163 -> 2963d59ed
HBASE-15229 Canary Tools should not call System.Exit on error (Vishal Khandelwal)
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/6bc35572
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/6bc35572
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/6bc35572
Branch: refs/heads/0.98
Commit: 6bc355721e1f5ec3934cf014f042702345d7e7b7
Parents: 62ce0e2
Author: Andrew Purtell <ap...@apache.org>
Authored: Wed Feb 10 10:19:49 2016 -0800
Committer: Andrew Purtell <ap...@apache.org>
Committed: Wed Feb 10 10:19:55 2016 -0800
----------------------------------------------------------------------
.../main/java/org/apache/hadoop/hbase/tool/Canary.java | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/6bc35572/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
index 3cc3b7b..d99bc18 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
@@ -590,9 +590,9 @@ public final class Canary implements Tool {
if (this.failOnError && monitor.hasError()) {
monitorThread.interrupt();
if (monitor.initialized) {
- System.exit(monitor.errorCode);
+ return monitor.errorCode;
} else {
- System.exit(INIT_ERROR_EXIT_CODE);
+ return INIT_ERROR_EXIT_CODE;
}
}
currentTimeLength = System.currentTimeMillis() - startTime;
@@ -601,17 +601,16 @@ public final class Canary implements Tool {
+ ") after timeout limit:" + this.timeout
+ " will be killed itself !!");
if (monitor.initialized) {
- System.exit(TIMEOUT_ERROR_EXIT_CODE);
+ return TIMEOUT_ERROR_EXIT_CODE;
} else {
- System.exit(INIT_ERROR_EXIT_CODE);
+ return INIT_ERROR_EXIT_CODE;
}
- break;
}
}
if (this.failOnError && monitor.finalCheckForErrors()) {
monitorThread.interrupt();
- System.exit(monitor.errorCode);
+ return monitor.errorCode;
}
} finally {
if (monitor != null) monitor.close();
@@ -623,7 +622,7 @@ public final class Canary implements Tool {
connection.close();
}
- return(monitor.errorCode);
+ return monitor.errorCode;
}
private void printUsageAndExit() {
[3/4] hbase git commit: HBASE-15229 Canary Tools should not call
System.Exit on error (Vishal Khandelwal)
Posted by ap...@apache.org.
HBASE-15229 Canary Tools should not call System.Exit on error (Vishal Khandelwal)
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/e85575f5
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/e85575f5
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/e85575f5
Branch: refs/heads/branch-1
Commit: e85575f5683e25fa8e44969ede68330358de63a4
Parents: 5fe081e
Author: Andrew Purtell <ap...@apache.org>
Authored: Wed Feb 10 10:19:49 2016 -0800
Committer: Andrew Purtell <ap...@apache.org>
Committed: Wed Feb 10 10:21:22 2016 -0800
----------------------------------------------------------------------
.../main/java/org/apache/hadoop/hbase/tool/Canary.java | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/e85575f5/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
index 061a14f..f4837bd 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
@@ -604,9 +604,9 @@ public final class Canary implements Tool {
if (this.failOnError && monitor.hasError()) {
monitorThread.interrupt();
if (monitor.initialized) {
- System.exit(monitor.errorCode);
+ return monitor.errorCode;
} else {
- System.exit(INIT_ERROR_EXIT_CODE);
+ return INIT_ERROR_EXIT_CODE;
}
}
currentTimeLength = System.currentTimeMillis() - startTime;
@@ -615,17 +615,16 @@ public final class Canary implements Tool {
+ ") after timeout limit:" + this.timeout
+ " will be killed itself !!");
if (monitor.initialized) {
- System.exit(TIMEOUT_ERROR_EXIT_CODE);
+ return TIMEOUT_ERROR_EXIT_CODE;
} else {
- System.exit(INIT_ERROR_EXIT_CODE);
+ return INIT_ERROR_EXIT_CODE;
}
- break;
}
}
if (this.failOnError && monitor.finalCheckForErrors()) {
monitorThread.interrupt();
- System.exit(monitor.errorCode);
+ return monitor.errorCode;
}
} finally {
if (monitor != null) monitor.close();
@@ -638,7 +637,7 @@ public final class Canary implements Tool {
if (choreService != null) {
choreService.shutdown();
}
- return(monitor.errorCode);
+ return monitor.errorCode;
}
private void printUsageAndExit() {
[2/4] hbase git commit: HBASE-15229 Canary Tools should not call
System.Exit on error (Vishal Khandelwal)
Posted by ap...@apache.org.
HBASE-15229 Canary Tools should not call System.Exit on error (Vishal Khandelwal)
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/2963d59e
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/2963d59e
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/2963d59e
Branch: refs/heads/master
Commit: 2963d59ed21de246390426cbdb57c160a4f65303
Parents: d533181
Author: Andrew Purtell <ap...@apache.org>
Authored: Wed Feb 10 10:19:49 2016 -0800
Committer: Andrew Purtell <ap...@apache.org>
Committed: Wed Feb 10 10:21:00 2016 -0800
----------------------------------------------------------------------
.../main/java/org/apache/hadoop/hbase/tool/Canary.java | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/2963d59e/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
index b2cca32..9248c71 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
@@ -607,9 +607,9 @@ public final class Canary implements Tool {
if (this.failOnError && monitor.hasError()) {
monitorThread.interrupt();
if (monitor.initialized) {
- System.exit(monitor.errorCode);
+ return monitor.errorCode;
} else {
- System.exit(INIT_ERROR_EXIT_CODE);
+ return INIT_ERROR_EXIT_CODE;
}
}
currentTimeLength = System.currentTimeMillis() - startTime;
@@ -618,17 +618,16 @@ public final class Canary implements Tool {
+ ") after timeout limit:" + this.timeout
+ " will be killed itself !!");
if (monitor.initialized) {
- System.exit(TIMEOUT_ERROR_EXIT_CODE);
+ return TIMEOUT_ERROR_EXIT_CODE;
} else {
- System.exit(INIT_ERROR_EXIT_CODE);
+ return INIT_ERROR_EXIT_CODE;
}
- break;
}
}
if (this.failOnError && monitor.finalCheckForErrors()) {
monitorThread.interrupt();
- System.exit(monitor.errorCode);
+ return monitor.errorCode;
}
} finally {
if (monitor != null) monitor.close();
@@ -641,7 +640,7 @@ public final class Canary implements Tool {
if (choreService != null) {
choreService.shutdown();
}
- return(monitor.errorCode);
+ return monitor.errorCode;
}
private void printUsageAndExit() {
[4/4] hbase git commit: HBASE-15229 Canary Tools should not call
System.Exit on error (Vishal Khandelwal)
Posted by ap...@apache.org.
HBASE-15229 Canary Tools should not call System.Exit on error (Vishal Khandelwal)
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/3b6c3057
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/3b6c3057
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/3b6c3057
Branch: refs/heads/branch-1.2
Commit: 3b6c3057d24ac1b72a2072ed2a4c34554e5c3a6d
Parents: 8a2cb16
Author: Andrew Purtell <ap...@apache.org>
Authored: Wed Feb 10 10:19:49 2016 -0800
Committer: Andrew Purtell <ap...@apache.org>
Committed: Wed Feb 10 10:21:28 2016 -0800
----------------------------------------------------------------------
.../main/java/org/apache/hadoop/hbase/tool/Canary.java | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/3b6c3057/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
index f21b6d2..26351ce 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
@@ -585,9 +585,9 @@ public final class Canary implements Tool {
if (this.failOnError && monitor.hasError()) {
monitorThread.interrupt();
if (monitor.initialized) {
- System.exit(monitor.errorCode);
+ return monitor.errorCode;
} else {
- System.exit(INIT_ERROR_EXIT_CODE);
+ return INIT_ERROR_EXIT_CODE;
}
}
currentTimeLength = System.currentTimeMillis() - startTime;
@@ -596,17 +596,16 @@ public final class Canary implements Tool {
+ ") after timeout limit:" + this.timeout
+ " will be killed itself !!");
if (monitor.initialized) {
- System.exit(TIMEOUT_ERROR_EXIT_CODE);
+ return TIMEOUT_ERROR_EXIT_CODE;
} else {
- System.exit(INIT_ERROR_EXIT_CODE);
+ return INIT_ERROR_EXIT_CODE;
}
- break;
}
}
if (this.failOnError && monitor.finalCheckForErrors()) {
monitorThread.interrupt();
- System.exit(monitor.errorCode);
+ return monitor.errorCode;
}
} finally {
if (monitor != null) monitor.close();
@@ -619,7 +618,7 @@ public final class Canary implements Tool {
if (choreService != null) {
choreService.shutdown();
}
- return(monitor.errorCode);
+ return monitor.errorCode;
}
private void printUsageAndExit() {