Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partit...

2017-08-06 Thread Murtadha Hubail (Code Review) 
Murtadha Hubail has uploaded a new change for review.

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

Change subject: [ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons 
reg/dereg
..

[ASTERIXDB-2019][CLUS] don't refresh cluster state on partitons reg/dereg

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

Details:
- Leave it to the caller when to refresh the cluster state
  after register/deregister of cluster partitions

Change-Id: I2bc5f86cedeb4728ccbb9811a36a4655a7786246
---
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/ClusterStateManager.java
1 file changed, 0 insertions(+), 7 deletions(-)


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

diff --git 
a/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/ClusterStateManager.java
 
b/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/ClusterStateManager.java
index 8156a23..1167816 100644
--- 
a/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/ClusterStateManager.java
+++ 
b/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/ClusterStateManager.java
@@ -374,12 +374,6 @@
 clusterPartitions.put(nodePartition.getPartitionId(), 
nodePartition);
 }
 node2PartitionsMap.put(nodeId, nodePartitions);
-//TODO fix exception propagation from refreshState
-try {
-refreshState();
-} catch (HyracksDataException e) {
-throw new AsterixException(e);
-}
 }
 
 @Override
@@ -394,7 +388,6 @@
 for (ClusterPartition nodePartition : nodePartitions) {
 clusterPartitions.remove(nodePartition.getPartitionId());
 }
-refreshState();
 }
 }
 

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

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