Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/2792/ (7/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/3340/ (1/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Analytics Compatibility Compilation Successful https://goo.gl/KzKq8B : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/3027/ (7/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Analytics Compatibility Compilation Successful https://goo.gl/87WHTN : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/2767/ (6/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/816/ (8/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/2801/ (9/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/2768/ (2/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Michael Blow has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/2415
Change subject: [NO ISSUE] Add Invoke.runUninterruptible
..
[NO ISSUE] Add Invoke.runUninterruptible
Add helper method to clear interruption and restore around invocation of an
action
Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6
---
M
hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
1 file changed, 33 insertions(+), 12 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/15/2415/1
diff --git
a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
index 9b356a0..bdaf479 100644
---
a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
+++
b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
@@ -39,7 +39,7 @@
* Executes the passed interruptible, retrying if the operation is
interrupted. Once the interruptible
* completes, the current thread will be re-interrupted, if the original
operation was interrupted.
*/
-public static void doUninterruptibly(Interruptible interruptible) {
+public static void doUninterruptibly(InterruptibleAction interruptible) {
boolean interrupted = false;
try {
while (true) {
@@ -58,10 +58,10 @@
}
/**
- * Executes the passed interruptible, retrying if the operation is
interrupted. Once the interruptible
+ * Executes the passed action, retrying if the operation is interrupted.
Once the interruptible
* completes, the current thread will be re-interrupted, if the original
operation was interrupted.
*/
-public static void doExUninterruptibly(ThrowingInterruptible
interruptible) throws Exception {
+public static void doExUninterruptibly(ThrowingAction interruptible)
throws Exception {
boolean interrupted = false;
try {
while (true) {
@@ -84,7 +84,7 @@
*
* @return true if the original operation was interrupted, otherwise false
*/
-public static boolean doUninterruptiblyGet(Interruptible interruptible) {
+public static boolean doUninterruptiblyGet(InterruptibleAction
interruptible) {
boolean interrupted = false;
while (true) {
try {
@@ -103,12 +103,12 @@
*
* @return true if the original operation was interrupted, otherwise false
*/
-public static boolean doExUninterruptiblyGet(ThrowingInterruptible
interruptible) throws Exception {
+public static boolean doExUninterruptiblyGet(Callable interruptible)
throws Exception {
boolean interrupted = false;
boolean success = false;
while (true) {
try {
-interruptible.run();
+interruptible.call();
success = true;
break;
} catch (InterruptedException e) { // NOSONAR- contract states
caller must handle
@@ -154,7 +154,7 @@
* {@link InterruptedException}. Once the interruptible completes, the
current thread will be re-interrupted, if
* the original operation was interrupted.
*/
-public static void doIoUninterruptibly(ThrowingIOInterruptible
interruptible) throws IOException {
+public static void doIoUninterruptibly(IOInterruptibleAction
interruptible) throws IOException {
boolean interrupted = false;
try {
while (true) {
@@ -175,7 +175,7 @@
}
@SuppressWarnings({ "squid:S1181", "squid:S1193" }) // catching Throwable,
instanceof of exception
-public static void tryWithCleanups(ThrowingInterruptible action,
ThrowingInterruptible... cleanups)
+public static void tryWithCleanups(ThrowingAction action,
ThrowingAction... cleanups)
throws Exception {
Throwable savedT = null;
boolean suppressedInterrupted = false;
@@ -184,7 +184,7 @@
} catch (Throwable t) {
savedT = t;
} finally {
-for (ThrowingInterruptible cleanup : cleanups) {
+for (ThrowingAction cleanup : cleanups) {
try {
cleanup.run();
} catch (Throwable t) {
@@ -212,18 +212,39 @@
}
}
+/**
+ * Runs the supplied action, after suspending any pending interruption.
An error will be logged if
+ * the action is itself interrupted.
+ */
+public static void runUninterruptible(ThrowingAction action) throws
Exception {
+boolean interrupted = Thread.interrupted();
+try {
+action.run();
+if (Thread.interrupted()) {
+throw new InterruptedException();
+}
+} catch (Interrup
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Michael Blow has uploaded a new patch set (#2). Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. [NO ISSUE] Add Invoke.runUninterruptible Add helper method to clear interruption and restore around invocation of an action Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 --- M hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java 1 file changed, 33 insertions(+), 13 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/15/2415/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/6759/ (3/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/817/ (4/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/2793/ (1/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/6760/ (8/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/2802/ (9/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Murtadha Hubail has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Code-Review+2 Integration-Tests+1 Verified+1 -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2750/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/2720/ (4/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/3026/ (2/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Contrib+1 BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2750/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Michael Blow has submitted this change and it was merged.
Change subject: [NO ISSUE] Add Invoke.runUninterruptible
..
[NO ISSUE] Add Invoke.runUninterruptible
Add helper method to clear interruption and restore around invocation of an
action
Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2415
Sonar-Qube: Jenkins
Reviewed-by: Murtadha Hubail
Integration-Tests: Murtadha Hubail
Tested-by: Murtadha Hubail
Contrib: Jenkins
---
M
hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
1 file changed, 33 insertions(+), 13 deletions(-)
Approvals:
Jenkins: No violations found;
Murtadha Hubail: Looks good to me, approved; Verified; Verified
diff --git
a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
index 9b356a0..ba4f82d 100644
---
a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
+++
b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/InvokeUtil.java
@@ -39,7 +39,7 @@
* Executes the passed interruptible, retrying if the operation is
interrupted. Once the interruptible
* completes, the current thread will be re-interrupted, if the original
operation was interrupted.
*/
-public static void doUninterruptibly(Interruptible interruptible) {
+public static void doUninterruptibly(InterruptibleAction interruptible) {
boolean interrupted = false;
try {
while (true) {
@@ -58,10 +58,10 @@
}
/**
- * Executes the passed interruptible, retrying if the operation is
interrupted. Once the interruptible
+ * Executes the passed action, retrying if the operation is interrupted.
Once the interruptible
* completes, the current thread will be re-interrupted, if the original
operation was interrupted.
*/
-public static void doExUninterruptibly(ThrowingInterruptible
interruptible) throws Exception {
+public static void doExUninterruptibly(ThrowingAction interruptible)
throws Exception {
boolean interrupted = false;
try {
while (true) {
@@ -84,7 +84,7 @@
*
* @return true if the original operation was interrupted, otherwise false
*/
-public static boolean doUninterruptiblyGet(Interruptible interruptible) {
+public static boolean doUninterruptiblyGet(InterruptibleAction
interruptible) {
boolean interrupted = false;
while (true) {
try {
@@ -103,12 +103,12 @@
*
* @return true if the original operation was interrupted, otherwise false
*/
-public static boolean doExUninterruptiblyGet(ThrowingInterruptible
interruptible) throws Exception {
+public static boolean doExUninterruptiblyGet(Callable interruptible)
throws Exception {
boolean interrupted = false;
boolean success = false;
while (true) {
try {
-interruptible.run();
+interruptible.call();
success = true;
break;
} catch (InterruptedException e) { // NOSONAR- contract states
caller must handle
@@ -154,7 +154,7 @@
* {@link InterruptedException}. Once the interruptible completes, the
current thread will be re-interrupted, if
* the original operation was interrupted.
*/
-public static void doIoUninterruptibly(ThrowingIOInterruptible
interruptible) throws IOException {
+public static void doIoUninterruptibly(IOInterruptibleAction
interruptible) throws IOException {
boolean interrupted = false;
try {
while (true) {
@@ -175,8 +175,7 @@
}
@SuppressWarnings({ "squid:S1181", "squid:S1193" }) // catching Throwable,
instanceof of exception
-public static void tryWithCleanups(ThrowingInterruptible action,
ThrowingInterruptible... cleanups)
-throws Exception {
+public static void tryWithCleanups(ThrowingAction action,
ThrowingAction... cleanups) throws Exception {
Throwable savedT = null;
boolean suppressedInterrupted = false;
try {
@@ -184,7 +183,7 @@
} catch (Throwable t) {
savedT = t;
} finally {
-for (ThrowingInterruptible cleanup : cleanups) {
+for (ThrowingAction cleanup : cleanups) {
try {
cleanup.run();
} catch (Throwable t) {
@@ -212,18 +211,39 @@
}
}
+/**
+ * Runs the supplied action, after suspending any pending interruption.
An error will be logged if
+ * the action is itself interrupted.
+ */
+public static void runUninterruptible(ThrowingAction action) throws
Exception {
+boolean
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/5723/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/3543/ (10/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/3282/ (11/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/3306/ (12/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/2721/ (6/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/3341/ (5/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/5723/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/8267/ (3/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] Add Invoke.runUninterruptible
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Add Invoke.runUninterruptible .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/8266/ (5/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2415 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f0a5dcb36bfc43168498b9a0a2a4afccedfc4f6 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
