----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14728/#review27744 -----------------------------------------------------------
Ship it! Ship It! - Kishore Gopalakrishna On Oct. 17, 2013, 9:38 p.m., Zhen Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14728/ > ----------------------------------------------------------- > > (Updated Oct. 17, 2013, 9:38 p.m.) > > > Review request for helix, Kanak Biscuitwala and Kishore Gopalakrishna. > > > 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 > ----- > > 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/manager/zk/HelixConnectionAdaptor.java > e69de29 > > 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/participant/statemachine/HelixStateModelFactory.java > e69de29 > > helix-core/src/main/java/org/apache/helix/participant/statemachine/HelixStateModelFactoryAdaptor.java > e69de29 > helix-core/src/test/java/org/apache/helix/AppTest.java e69de29 > > Diff: https://reviews.apache.org/r/14728/diff/ > > > Testing > ------- > > AppTest > > > Thanks, > > Zhen Zhang > >
