-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14728/
-----------------------------------------------------------

(Updated Nov. 6, 2013, 1:53 a.m.)


Review request for helix, Kanak Biscuitwala and Kishore Gopalakrishna.


Changes
-------

updated diff


Repository: helix-git


Description
-------

replace HelixManager with HelixConnection so we can reuse connections. major 
interfaces are:

HelixConnection
HelixConnectionStateListener

HelixService
HelixRole
HelixParticipant
HelixController
HelixAutoController

AppTest shows an example of using new APIs to setup and start a cluster


Diffs (updated)
-----

  helix-core/src/main/java/org/apache/helix/HelixAutoController.java e69de29 
  helix-core/src/main/java/org/apache/helix/HelixConnection.java e69de29 
  helix-core/src/main/java/org/apache/helix/HelixConnectionStateListener.java 
e69de29 
  helix-core/src/main/java/org/apache/helix/HelixController.java e69de29 
  helix-core/src/main/java/org/apache/helix/HelixParticipant.java e69de29 
  helix-core/src/main/java/org/apache/helix/HelixRole.java e69de29 
  helix-core/src/main/java/org/apache/helix/HelixService.java e69de29 
  helix-core/src/main/java/org/apache/helix/api/accessor/ClusterAccessor.java 
85b8432 
  
helix-core/src/main/java/org/apache/helix/manager/zk/HelixConnectionAdaptor.java
 e69de29 
  helix-core/src/main/java/org/apache/helix/manager/zk/ZkBaseDataAccessor.java 
0b112cd 
  
helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixAutoController.java 
e69de29 
  helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixConnection.java 
e69de29 
  helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixController.java 
e69de29 
  
helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixLeaderElection.java 
e69de29 
  helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixParticipant.java 
e69de29 
  helix-core/src/main/java/org/apache/helix/monitoring/StatusDumpTask.java 
e69de29 
  
helix-core/src/main/java/org/apache/helix/participant/HelixStateMachineEngine.java
 4e4fdf6 
  helix-core/src/main/java/org/apache/helix/participant/StateMachineEngine.java 
d11b3cc 
  
helix-core/src/main/java/org/apache/helix/participant/statemachine/HelixStateModelFactory.java
 e69de29 
  
helix-core/src/main/java/org/apache/helix/participant/statemachine/HelixStateModelFactoryAdaptor.java
 e69de29 
  
helix-core/src/main/java/org/apache/helix/participant/statemachine/StateModelFactory.java
 4d8e598 
  
helix-core/src/test/java/org/apache/helix/integration/TestHelixConnection.java 
e69de29 

Diff: https://reviews.apache.org/r/14728/diff/


Testing
-------

AppTest


Thanks,

Zhen Zhang

Reply via email to