----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15895/#review29752 -----------------------------------------------------------
Ship it! Ship It! - Kishore Gopalakrishna On Dec. 2, 2013, 10:59 p.m., Kanak Biscuitwala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15895/ > ----------------------------------------------------------- > > (Updated Dec. 2, 2013, 10:59 p.m.) > > > Review request for helix, Zhen Zhang and Kishore Gopalakrishna. > > > Bugs: HELIX-319 > > > Repository: helix-git > > > Description > ------- > > commit 084ce23546ba6f8c2fe9c5826fe360ce85d09527 > Author: Kanak Biscuitwala <[email protected]> > Date: Wed Nov 27 16:20:36 2013 -0800 > > [HELIX-319] Plug Riemann server into Helix, config management, first take > > :100644 100644 91ec809... 5bb6558... M > helix-core/src/main/java/org/apache/helix/HelixAutoController.java > :100644 100644 9565cfb... d55a0a1... M > helix-core/src/main/java/org/apache/helix/HelixController.java > :100644 100644 54e9943... 05ccde2... M > helix-core/src/main/java/org/apache/helix/HelixManager.java > :100644 100644 ebd7a35... 228863a... M > helix-core/src/main/java/org/apache/helix/PropertyKey.java > :100644 100644 9836020... bcc68b6... M > helix-core/src/main/java/org/apache/helix/manager/zk/DistributedLeaderElection.java > :100644 100644 e52427c... 33dcc06... M > helix-core/src/main/java/org/apache/helix/manager/zk/HelixConnectionAdaptor.java > :100644 100644 b71304a... 1beb7f3... M > helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixManager.java > :100644 100644 136d47e... 75fa69f... M > helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixAutoController.java > :100644 100644 f116c76... 0e140a5... M > helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixController.java > :100644 100644 77da158... d1bdcb0... M > helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixLeaderElection.java > :100644 100644 95c9848... 65847d5... M > helix-core/src/main/java/org/apache/helix/model/HelixConfigScope.java > :100644 100644 e9348ec... 21d045b... M > helix-core/src/main/java/org/apache/helix/model/LiveInstance.java > :000000 100644 0000000... b9aa17c... A > helix-core/src/main/java/org/apache/helix/model/MonitoringConfig.java > :100644 100644 d65ffd3... 87cabcf... M > helix-core/src/main/java/org/apache/helix/model/builder/HelixConfigScopeBuilder.java > :000000 100644 0000000... 8ea6c43... A > helix-core/src/main/java/org/apache/helix/monitoring/MonitoringService.java > :100644 100644 919eb00... 26843ee... M > helix-core/src/test/java/org/apache/helix/Mocks.java > :100644 100644 fc9b7d5... aa500e3... M > helix-core/src/test/java/org/apache/helix/controller/stages/DummyClusterManager.java > :100644 100644 d97b22a... b7f2de6... M > helix-core/src/test/java/org/apache/helix/participant/MockZKHelixManager.java > :000000 100644 0000000... 2001d31... A helix-monitor-server/DISCLAIMER > :000000 100644 0000000... 413913f... A helix-monitor-server/LICENSE > :000000 100644 0000000... e070e15... A helix-monitor-server/NOTICE > :000000 100644 0000000... 36e840d... A helix-monitor-server/pom.xml > :000000 100644 0000000... c2d08a1... A > helix-monitor-server/src/assemble/assembly.xml > :000000 100644 0000000... 4b3dc31... A > helix-monitor-server/src/main/config/log4j.properties > :000000 100644 0000000... e95e9bb... A > helix-monitor-server/src/main/java/org/apache/helix/monitoring/RiemannMonitoringService.java > :000000 100644 0000000... 08c3bce... A > helix-monitor-server/src/main/resources/riemann.config > :000000 100644 0000000... 90910aa... A > helix-monitor-server/src/test/conf/testng.xml > :000000 100644 0000000... ee64ca2... A > helix-monitor-server/src/test/java/org/apache/helix/monitoring/BasicMonitoringTest.java > :100644 100644 010023c... c220e2b... M pom.xml > > > Diffs > ----- > > > helix-admin-webapp/src/main/java/org/apache/helix/webapp/resources/ClusterResource.java > b22d801 > > helix-admin-webapp/src/main/java/org/apache/helix/webapp/resources/ControllerResource.java > ea7be42 > > helix-admin-webapp/src/main/java/org/apache/helix/webapp/resources/SchedulerTasksResource.java > 5803e98 > > helix-admin-webapp/src/test/java/org/apache/helix/tools/TestHelixAdminScenariosRest.java > 6a0e331 > helix-core/src/main/java/org/apache/helix/HelixAutoController.java 91ec809 > helix-core/src/main/java/org/apache/helix/HelixController.java 9565cfb > helix-core/src/main/java/org/apache/helix/HelixManager.java 54e9943 > helix-core/src/main/java/org/apache/helix/PropertyKey.java ebd7a35 > helix-core/src/main/java/org/apache/helix/api/accessor/ClusterAccessor.java > b01a3ec > > helix-core/src/main/java/org/apache/helix/api/accessor/ControllerAccessor.java > 609e458 > > helix-core/src/main/java/org/apache/helix/controller/GenericHelixController.java > eec745e > > helix-core/src/main/java/org/apache/helix/manager/zk/DistributedLeaderElection.java > 9836020 > > helix-core/src/main/java/org/apache/helix/manager/zk/HelixConnectionAdaptor.java > e52427c > > helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixDataAccessor.java > 471530c > helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixManager.java > b71304a > > helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixAutoController.java > 136d47e > helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixController.java > f116c76 > > helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixLeaderElection.java > 77da158 > helix-core/src/main/java/org/apache/helix/model/HelixConfigScope.java > 95c9848 > helix-core/src/main/java/org/apache/helix/model/Leader.java PRE-CREATION > helix-core/src/main/java/org/apache/helix/model/LiveInstance.java e9348ec > helix-core/src/main/java/org/apache/helix/model/MonitoringConfig.java > PRE-CREATION > > helix-core/src/main/java/org/apache/helix/model/builder/HelixConfigScopeBuilder.java > d65ffd3 > helix-core/src/main/java/org/apache/helix/monitoring/MonitoringServer.java > PRE-CREATION > > helix-core/src/main/java/org/apache/helix/monitoring/MonitoringServerOwner.java > PRE-CREATION > > helix-core/src/main/java/org/apache/helix/participant/DistClusterControllerElection.java > 0e8c6fd > helix-core/src/test/java/org/apache/helix/Mocks.java 919eb00 > > helix-core/src/test/java/org/apache/helix/controller/stages/DummyClusterManager.java > fc9b7d5 > > helix-core/src/test/java/org/apache/helix/integration/TestDistributedCMMain.java > 8135191 > > helix-core/src/test/java/org/apache/helix/integration/TestDistributedClusterController.java > f3def23 > > helix-core/src/test/java/org/apache/helix/integration/TestStandAloneCMMain.java > 6eb7a8c > > helix-core/src/test/java/org/apache/helix/integration/ZkIntegrationTestBase.java > 9188e61 > > helix-core/src/test/java/org/apache/helix/integration/manager/TestConsecutiveZkSessionExpiry.java > 8625c0c > > helix-core/src/test/java/org/apache/helix/integration/manager/TestDistributedControllerManager.java > aa00a8d > helix-core/src/test/java/org/apache/helix/manager/zk/TestZkFlapping.java > 79c74f5 > > helix-core/src/test/java/org/apache/helix/manager/zk/TestZkHelixAutoController.java > 28b1477 > > helix-core/src/test/java/org/apache/helix/manager/zk/TestZkHelixController.java > 0127edb > > helix-core/src/test/java/org/apache/helix/participant/MockZKHelixManager.java > d97b22a > > helix-core/src/test/java/org/apache/helix/participant/TestDistControllerElection.java > 28068ec > helix-core/src/test/java/org/apache/helix/tools/TestHelixAdminCli.java > 4d7f93c > helix-monitor-server/.gitignore PRE-CREATION > helix-monitor-server/DISCLAIMER PRE-CREATION > helix-monitor-server/LICENSE PRE-CREATION > helix-monitor-server/NOTICE PRE-CREATION > helix-monitor-server/pom.xml PRE-CREATION > helix-monitor-server/src/assemble/assembly.xml PRE-CREATION > helix-monitor-server/src/main/config/log4j.properties PRE-CREATION > > helix-monitor-server/src/main/java/org/apache/helix/monitoring/RiemannMonitoringServer.java > PRE-CREATION > helix-monitor-server/src/main/resources/riemann.config PRE-CREATION > helix-monitor-server/src/test/conf/testng.xml PRE-CREATION > > helix-monitor-server/src/test/java/org/apache/helix/monitoring/BasicMonitoringTest.java > PRE-CREATION > pom.xml 010023c > > Diff: https://reviews.apache.org/r/15895/diff/ > > > Testing > ------- > > New test verifies that the server registers. > > > Thanks, > > Kanak Biscuitwala > >
