----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13754/#review25493 -----------------------------------------------------------
Ship it! I think this is a good start. helix-core/src/main/java/org/apache/helix/api/Participant.java <https://reviews.apache.org/r/13754/#comment49943> May be nice to note for this and other functions that the returned collection is immutable. - Kanak Biscuitwala On Aug. 24, 2013, 1:33 a.m., Zhen Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13754/ > ----------------------------------------------------------- > > (Updated Aug. 24, 2013, 1:33 a.m.) > > > Review request for helix, Kanak Biscuitwala, Kishore Gopalakrishna, and Shi > Lu. > > > Repository: helix-git > > > Description > ------- > > Initial change for refactor helix model. Roughly follow the design at: > https://cwiki.apache.org/confluence/display/HELIX/API+Redesign > > An example is shown in ClusterReader where using a Cluster instance, we can > retrieve any information needed for the rebalancer. The change is incomplete. > > > Diffs > ----- > > helix-core/src/main/java/org/apache/helix/api/Cluster.java e69de29 > helix-core/src/main/java/org/apache/helix/api/ClusterReader.java e69de29 > helix-core/src/main/java/org/apache/helix/api/Controller.java e69de29 > helix-core/src/main/java/org/apache/helix/api/CurState.java e69de29 > helix-core/src/main/java/org/apache/helix/api/ExtView.java e69de29 > helix-core/src/main/java/org/apache/helix/api/HelixVersion.java e69de29 > helix-core/src/main/java/org/apache/helix/api/Msg.java e69de29 > helix-core/src/main/java/org/apache/helix/api/Participant.java e69de29 > helix-core/src/main/java/org/apache/helix/api/Partition.java e69de29 > helix-core/src/main/java/org/apache/helix/api/Resource.java e69de29 > helix-core/src/main/java/org/apache/helix/api/RunningInstance.java e69de29 > helix-core/src/main/java/org/apache/helix/api/Spectator.java e69de29 > > Diff: https://reviews.apache.org/r/13754/diff/ > > > Testing > ------- > > > Thanks, > > Zhen Zhang > >
