-----------------------------------------------------------
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.


Changes
-------

per discussion with Kanak, make Cluster instance read-only by using google 
guava immutable collection objects. mostly follow the following logical 
hierarchy:

- CLUSTER
      - PARTICIPANT(*)
         - CONFIG
         - MESSAGE
      - SPECTATOR(*)
      - CONTROLLER(*)
      - RESOURCE
         - CONFIG
         - IDEALSTATE
         - EXTERNALVIEW
         - CURRENTSTATE


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 (updated)
-----

  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

Reply via email to