Yingyi Bu has posted comments on this change.

Change subject: Remove ICCContext
......................................................................


Patch Set 5:

>>I am aware of this Yingyi, 
>>but the map is updated only when the 
>> cluster is not in an Active State.
>> In that sense, when know it is okay to get the map.

I don't think that's the right thing (the master) -- cluster membership should 
be maintained dynamically.  In addition,  "Active State", i.e., ClusterState is 
an asterixdb concept.  Hyracks doesn't know anything of this and shouldn't 
reply on this.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6f6a769741f14e91bcd4b970b4a022c0a453d380
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-Reviewer: Yingyi Bu <[email protected]>
Gerrit-Reviewer: abdullah alamoudi <[email protected]>
Gerrit-HasComments: No

Reply via email to