Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-14 Thread Murtadha Hubail (Code Review)
Murtadha Hubail has submitted this change and it was merged. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..

[NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Ignore received node heartbeats on CC when the CC executor
  has not been initialized since nodes are not registered yet.

Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Reviewed-on: https://asterix-gerrit.ics.uci.edu/3395
Tested-by: Jenkins 
Integration-Tests: Jenkins 
Contrib: Jenkins 
Reviewed-by: Michael Blow 
---
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
1 file changed, 10 insertions(+), 3 deletions(-)

Approvals:
  Jenkins: Verified; ; Verified
  Anon. E. Moose (1000171):
  Michael Blow: Looks good to me, approved

Objections:
  Jenkins: Violations found



diff --git 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
index 78ed9b9..e022dfe 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
@@ -19,6 +19,7 @@
 package org.apache.hyracks.control.cc;

 import java.util.Map;
+import java.util.concurrent.ExecutorService;

 import org.apache.hyracks.api.client.NodeControllerInfo;
 import org.apache.hyracks.control.cc.work.ApplicationMessageWork;
@@ -73,9 +74,7 @@
 ccs.getWorkQueue().schedule(new 
UnregisterNodeWork(ccs.getNodeManager(), unf.getNodeId()));
 break;
 case NODE_HEARTBEAT:
-CCNCFunctions.NodeHeartbeatFunction nhf = 
(CCNCFunctions.NodeHeartbeatFunction) fn;
-ccs.getExecutor().execute(
-new NodeHeartbeatWork(ccs, nhf.getNodeId(), 
nhf.getHeartbeatData(), nhf.getNcAddress()));
+processNodeHeartbeat(ccs, fn);
 break;
 case NOTIFY_JOBLET_CLEANUP:
 CCNCFunctions.NotifyJobletCleanupFunction njcf = 
(CCNCFunctions.NotifyJobletCleanupFunction) fn;
@@ -170,4 +169,12 @@
 LOGGER.warn("Unknown function: " + fn.getFunctionId());
 }
 }
+
+private static void processNodeHeartbeat(ClusterControllerService ccs, 
CCNCFunctions.Function fn) {
+final ExecutorService executor = ccs.getExecutor();
+if (executor != null) {
+CCNCFunctions.NodeHeartbeatFunction nhf = 
(CCNCFunctions.NodeHeartbeatFunction) fn;
+executor.execute(new NodeHeartbeatWork(ccs, nhf.getNodeId(), 
nhf.getHeartbeatData(), nhf.getNcAddress()));
+}
+}
 }

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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 3
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-14 Thread Michael Blow (Code Review)
Michael Blow has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 18:20:19 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-14 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2: Contrib+1

BAD Compatibility Tests Successful

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 12:11:54 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-14 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 11:55:02 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Anon. E. Moose (Code Review)
Anon. E. Moose (1000171) has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2: Contrib+1

Analytics Compatibility Tests Successful
https://cbjenkins.page.link/g1L6EDzKjT11Nnaj7 : SUCCESS


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 04:05:27 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 03:12:38 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/1140/
 (16/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 02:10:57 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 02:00:48 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2: Contrib-2

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/4406/ : UNSTABLE


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:59:19 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3783/ 
(15/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:54:30 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/6354/ (13/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:50:54 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5994/ 
(14/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:54:20 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9730/ (10/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:47:23 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/1048/
 (12/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:49:01 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/11261/ (5/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:41 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Anon. E. Moose (Code Review)
Anon. E. Moose (1000171) has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Analytics Compatibility Compilation Successful
https://cbjenkins.page.link/QsTw2s99orGcn3qm7 : SUCCESS


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:48:06 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:48:11 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/950/ (9/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:47:20 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:45:36 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5788/ 
(7/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:44:02 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Murtadha Hubail (Code Review)
Hello Anon. E. Moose (1000171), Till Westmann, Jenkins, Michael Blow,

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

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

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

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..

[NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Ignore received node heartbeats on CC when the CC executor
  has not been initialized since nodes are not registered yet.

Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
---
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
1 file changed, 10 insertions(+), 3 deletions(-)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5771/
 (6/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:43:59 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/6144/ 
(8/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:44:32 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5751/ 
(1/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:43:46 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/11263/ (5/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:43:58 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:43:53 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-spidersilk-tests/712/ 
(2/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:43:50 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ssl-compression/553/ 
(4/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:43:55 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1: Contrib+1

BAD Compatibility Tests Successful

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Comment-Date: Tue, 14 May 2019 01:42:47 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Anon. E. Moose (Code Review)
Anon. E. Moose (1000171) has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Analytics Compatibility Compilation Successful
https://cbjenkins.page.link/yPDQ44LBDdrg2dxi7 : SUCCESS


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:30:36 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/948/ (11/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:50 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5749/ 
(4/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:40 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/6142/ 
(14/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:56 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/1046/
 (8/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:45 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Murtadha Hubail (Code Review)
Murtadha Hubail has uploaded this change for review. ( 
https://asterix-gerrit.ics.uci.edu/3395


Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..

[NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Ignore received node heartbeats on CC when the CC has not
  been initialized since nodes are not registered yet.

Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
---
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
1 file changed, 9 insertions(+), 3 deletions(-)



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

diff --git 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
index 78ed9b9..97e05da 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/ClusterControllerIPCI.java
@@ -19,6 +19,7 @@
 package org.apache.hyracks.control.cc;

 import java.util.Map;
+import java.util.concurrent.ExecutorService;

 import org.apache.hyracks.api.client.NodeControllerInfo;
 import org.apache.hyracks.control.cc.work.ApplicationMessageWork;
@@ -73,9 +74,7 @@
 ccs.getWorkQueue().schedule(new 
UnregisterNodeWork(ccs.getNodeManager(), unf.getNodeId()));
 break;
 case NODE_HEARTBEAT:
-CCNCFunctions.NodeHeartbeatFunction nhf = 
(CCNCFunctions.NodeHeartbeatFunction) fn;
-ccs.getExecutor().execute(
-new NodeHeartbeatWork(ccs, nhf.getNodeId(), 
nhf.getHeartbeatData(), nhf.getNcAddress()));
+processNodeHeartbeat(ccs, 
(CCNCFunctions.NodeHeartbeatFunction) fn);
 break;
 case NOTIFY_JOBLET_CLEANUP:
 CCNCFunctions.NotifyJobletCleanupFunction njcf = 
(CCNCFunctions.NotifyJobletCleanupFunction) fn;
@@ -170,4 +169,11 @@
 LOGGER.warn("Unknown function: " + fn.getFunctionId());
 }
 }
+
+private static void processNodeHeartbeat(ClusterControllerService ccs, 
CCNCFunctions.NodeHeartbeatFunction fn) {
+final ExecutorService executor = ccs.getExecutor();
+if (executor != null) {
+executor.execute(new NodeHeartbeatWork(ccs, fn.getNodeId(), 
fn.getHeartbeatData(), fn.getNcAddress()));
+}
+}
 }

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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:47 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:39 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:26:15 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5769/
 (16/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:24:00 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3781/ 
(13/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:54 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:26:34 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/1138/
 (10/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:48 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-spidersilk-tests/710/ 
(15/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:58 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ssl-compression/551/ 
(3/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:39 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:53 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/6352/ (7/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:44 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:42 +
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App Initialization

2019-05-13 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3395 )

Change subject: [NO ISSUE][CLUS] Ignore Received Heartbeats Before App 
Initialization
..


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5685/ 
(1/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65ef92ae3179214f1efecd1bb44da4772b7a3dd9
Gerrit-Change-Number: 3395
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Comment-Date: Tue, 14 May 2019 01:23:38 +
Gerrit-HasComments: No