Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4653/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Anon. E. Moose (Code Review)]

Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Analytics Compatibility Tests Failed Build 2987
: UNSTABLE

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4653/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/2313/ 
(11/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2167/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack

PLEASE REVIEW CAREFULLY AND LOOK FOR API CHANGES!

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Michael Blow (Code Review)]

Michael Blow has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4: Code-Review+2 Verified+1

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Michael Blow (Code Review)]

Michael Blow has submitted this change and it was merged.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


[NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is complete

- close NC IPC manager after sending shutdown notifcation to CC

Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2159
Reviewed-by: Michael Blow 
Tested-by: Michael Blow 
---
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InvokeUtil.java
M 
asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java
M asterixdb/asterix-yarn/src/main/java/org/apache/asterix/aoya/Utils.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/NotifyShutdownWork.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerIPCI.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/task/ShutdownTask.java
7 files changed, 97 insertions(+), 52 deletions(-)

Approvals:
  Michael Blow: Looks good to me, approved; Verified



diff --git 
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InterruptUtil.java
 
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InvokeUtil.java
similarity index 75%
rename from 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InterruptUtil.java
rename to 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InvokeUtil.java
index 4c65c66..c7ac0f4 100644
--- 
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InterruptUtil.java
+++ 
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InvokeUtil.java
@@ -18,7 +18,16 @@
  */
 package org.apache.asterix.common.utils;
 
-public class InterruptUtil {
+import java.io.IOException;
+import java.util.concurrent.Callable;
+import java.util.concurrent.TimeUnit;
+import java.util.logging.Level;
+import java.util.logging.Logger;
+
+public class InvokeUtil {
+
+private static final Logger LOGGER = 
Logger.getLogger(InvokeUtil.class.getName());
+
 /**
  * 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.
@@ -82,7 +91,7 @@
 }
 
 /**
- * Executes the passed interruptible, retrying if the operation is 
interrupted.  If the operation throws an
+ * Executes the passed interruptible, retrying if the operation is 
interrupted. If the operation throws an
  * exception after being previously interrupted, the current thread will 
be re-interrupted.
  *
  * @return true if the original operation was interrupted, otherwise false
@@ -106,6 +115,33 @@
 return interrupted;
 }
 
+public static boolean retryLoop(long duration, TimeUnit durationUnit, long 
delay, TimeUnit delayUnit,
+Callable function) throws IOException {
+long endTime = System.nanoTime() + durationUnit.toNanos(duration);
+boolean first = true;
+while (endTime - System.nanoTime() > 0) {
+if (first) {
+first = false;
+} else {
+try {
+delayUnit.sleep(delay);
+} catch (InterruptedException e) {
+Thread.currentThread().interrupt();
+return false;
+}
+}
+try {
+if (function.call()) {
+return true;
+}
+} catch (Exception e) {
+// ignore, retry after delay
+LOGGER.log(Level.FINE, "Ignoring exception on retryLoop 
attempt, will retry after delay", e);
+}
+}
+return false;
+}
+
 @FunctionalInterface
 public interface Interruptible {
 void run() throws InterruptedException;
@@ -115,4 +151,5 @@
 public interface ThrowingInterruptible {
 void run() throws Exception; // NOSONAR
 }
+
 }
diff --git 
a/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java
 
b/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java
index 4d671f3..dd0a5c7 100644
--- 
a/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java
+++ 

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2502/ (10/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1956/ 
(2/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/2290/
 (3/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Michael Blow (Code Review)]

Hello Anon. E. Moose #1000171, Jenkins, Murtadha Hubail,

I'd like you to reexamine a change.  Please visit

https://asterix-gerrit.ics.uci.edu/2159

to look at the new patch set (#4).

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..

[NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is complete

- close NC IPC manager after sending shutdown notifcation to CC

Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
---
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InvokeUtil.java
M 
asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java
M asterixdb/asterix-yarn/src/main/java/org/apache/asterix/aoya/Utils.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/NotifyShutdownWork.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerIPCI.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/task/ShutdownTask.java
7 files changed, 97 insertions(+), 52 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/59/2159/4
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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 

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1641/
 (8/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1622/ 
(4/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5676/ (9/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-rebalance-cancellation/821/
 (6/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1634/ 
(5/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/2249/ (1/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/7174/ (7/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3: -Integration-Tests

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4652/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
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][HYR] Notify CC of NC shutdown only after shutdown...

[Murtadha Hubail (Code Review)]

Murtadha Hubail has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3: Code-Review+2

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4648/ 
: UNSTABLE

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
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][HYR] Notify CC of NC shutdown only after shutdown...

[Anon. E. Moose (Code Review)]

Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3: Contrib+2

Analytics Compatibility Tests Successful Build 2983
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3: Contrib+2

BAD Compatibility Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2163/ : SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Michael Blow (Code Review)]

Michael Blow has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/2159/3/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InvokeUtil.java
File 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InvokeUtil.java:

PS3, Line 154: 
 : @FunctionalInterface
 : public interface IOCallable {
 : V call() throws IOException;
 : }
 : 
remove


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: Yes

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2163/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4648/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/2309/ 
(11/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2499/ (10/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/2286/
 (9/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack

PLEASE REVIEW CAREFULLY AND LOOK FOR API CHANGES!

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/7171/ (7/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/2246/ (3/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5673/ (2/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1953/ 
(6/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1631/ 
(8/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1619/ 
(4/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1638/
 (1/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-rebalance-cancellation/817/
 (5/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Michael Blow (Code Review)]

Hello Jenkins, Murtadha Hubail,

I'd like you to reexamine a change.  Please visit

https://asterix-gerrit.ics.uci.edu/2159

to look at the new patch set (#3).

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..

[NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is complete

- close NC IPC manager after sending shutdown notifcation to CC

Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
---
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/InvokeUtil.java
M 
asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java
M asterixdb/asterix-yarn/src/main/java/org/apache/asterix/aoya/Utils.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/NotifyShutdownWork.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerIPCI.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/task/ShutdownTask.java
7 files changed, 102 insertions(+), 52 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/59/2159/3
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/2280/
 (11/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/2279/
 (11/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2493/ (11/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/7163/ (11/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4637/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4637/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/2294/ 
(11/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2486/ (10/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/2274/
 (9/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Murtadha Hubail (Code Review)]

Murtadha Hubail has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2: Code-Review+2

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1607/ 
(7/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-rebalance-cancellation/804/
 (1/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1619/ 
(4/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5661/ (6/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/7158/ (8/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1941/ 
(2/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1626/
 (3/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/2234/ (5/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Michael Blow (Code Review)]

Michael Blow has uploaded a new patch set (#2).

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..

[NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is complete

- close NC IPC manager after sending shutdown notifcation to CC

Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
---
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/NotifyShutdownWork.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerIPCI.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/task/ShutdownTask.java
4 files changed, 16 insertions(+), 22 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/59/2159/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1625/
 (7/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5660/ (8/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Michael Blow (Code Review)]

Michael Blow has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/2159

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..

[NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is complete

- close NC IPC manager after sending shutdown notifcation to CC

Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
---
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/NotifyShutdownWork.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerIPCI.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/task/ShutdownTask.java
4 files changed, 16 insertions(+), 22 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/59/2159/1

diff --git 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/NotifyShutdownWork.java
 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/NotifyShutdownWork.java
index 5119022..83cbb91 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/NotifyShutdownWork.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/NotifyShutdownWork.java
@@ -27,9 +27,9 @@
 
 public class NotifyShutdownWork extends SynchronizableWork {
 
+private static final Logger LOGGER = 
Logger.getLogger(NotifyShutdownWork.class.getName());
 private final ClusterControllerService ccs;
 private final String nodeId;
-private static Logger LOGGER = 
Logger.getLogger(NotifyShutdownWork.class.getName());
 
 public NotifyShutdownWork(ClusterControllerService ccs, String nodeId) {
 this.ccs = ccs;
@@ -41,8 +41,12 @@
 public void doRun() {
 // Triggered remotely by a NC to notify that the NC is shutting down.
 ShutdownRun sRun = ccs.getShutdownRun();
-LOGGER.info("Received shutdown acknowledgement from NC ID:" + nodeId);
-sRun.notifyShutdown(nodeId);
+if (sRun != null) {
+LOGGER.info("Received shutdown acknowledgement from node " + 
nodeId);
+sRun.notifyShutdown(nodeId);
+} else {
+LOGGER.info("Received unsolicted shutdown notification from node " 
+ nodeId);
+}
 }
 
 }
diff --git 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerIPCI.java
 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerIPCI.java
index c54f153..b220039 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerIPCI.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerIPCI.java
@@ -122,7 +122,7 @@
 
 case SHUTDOWN_REQUEST:
 final CCNCFunctions.ShutdownRequestFunction sdrf = 
(CCNCFunctions.ShutdownRequestFunction) fn;
-ncs.getExecutor().submit(new ShutdownTask(ncs, 
sdrf.isTerminateNCService()));
+ncs.getExecutor().submit(new 
ShutdownTask(sdrf.isTerminateNCService()));
 return;
 
 case THREAD_DUMP_REQUEST:
diff --git 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java
 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java
index 9dd9536..a3a9ac5 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java
@@ -409,6 +409,13 @@
 heartbeatThread.interrupt();
 heartbeatThread.join(1000); // give it 1s to stop gracefully
 }
+try {
+ccs.notifyShutdown(id);
+} catch (Exception e) {
+LOGGER.log(Level.WARNING, "Exception notifying CC of 
shutdown", e);
+}
+ipc.stop();
+
 LOGGER.log(Level.INFO, "Stopped NodeControllerService");
 } else {
 LOGGER.log(Level.SEVERE, "Duplicate shutdown call; original: " + 
Arrays.toString(shutdownCallStack),
diff --git 

Change in asterixdb[master]: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1940/ 
(6/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/2233/ (1/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1606/ 
(3/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-rebalance-cancellation/803/
 (5/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1618/ 
(2/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
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][HYR] Notify CC of NC shutdown only after shutdown...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: [NO ISSUE][HYR] Notify CC of NC shutdown only after shutdown is 
complete
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/7157/ (4/11)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2159
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idde1f69a0e0a9a948898d9271441ca95485b77f4
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No