----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24332/#review49943 -----------------------------------------------------------
Ship it! Ship It! - Kanak Biscuitwala On Aug. 7, 2014, 10:50 a.m., Zhen Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24332/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2014, 10:50 a.m.) > > > Review request for helix, Kanak Biscuitwala and Kishore Gopalakrishna. > > > Bugs: HELIX-484 and HELIX-486 > > > Repository: helix-git > > > Description > ------- > > Remove CallbackHandler/ZkCallbackHandler code duplication > Remove StateModelFactory/HelixStateModelFactory code duplication > > > Diffs > ----- > > > helix-admin-webapp/src/test/java/org/apache/helix/webapp/TestResetInstance.java > a9ecaa0 > > helix-admin-webapp/src/test/java/org/apache/helix/webapp/TestResetResource.java > a54b0a3 > helix-agent/src/main/java/org/apache/helix/agent/AgentStateModel.java > d227ac3 > > helix-agent/src/main/java/org/apache/helix/agent/AgentStateModelFactory.java > 69d45ae > > helix-core/src/main/java/org/apache/helix/api/StateTransitionHandlerFactory.java > 45f56e5 > helix-core/src/main/java/org/apache/helix/api/TransitionHandler.java > 9717340 > helix-core/src/main/java/org/apache/helix/api/id/StateModelDefId.java > 7c84f0f > > helix-core/src/main/java/org/apache/helix/controller/HelixControllerMain.java > 6aa3ab9 > > helix-core/src/main/java/org/apache/helix/manager/zk/ZkHelixParticipant.java > af50eb7 > > helix-core/src/main/java/org/apache/helix/messaging/handling/HelixStateTransitionHandler.java > 1bb6506 > > helix-core/src/main/java/org/apache/helix/participant/CustomCodeInvoker.java > 6c96629 > > helix-core/src/main/java/org/apache/helix/participant/DistClusterControllerStateModel.java > 0c2eb7c > > helix-core/src/main/java/org/apache/helix/participant/DistClusterControllerStateModelFactory.java > a367c81 > > helix-core/src/main/java/org/apache/helix/participant/GenericLeaderStandbyModel.java > 3866cf5 > > helix-core/src/main/java/org/apache/helix/participant/GenericLeaderStandbyStateModelFactory.java > 51c91cc > > helix-core/src/main/java/org/apache/helix/participant/HelixCustomCodeRunner.java > 2f169cc > > helix-core/src/main/java/org/apache/helix/participant/HelixStateMachineEngine.java > 95afb70 > > helix-core/src/main/java/org/apache/helix/participant/StateMachineEngine.java > abb7d81 > > helix-core/src/main/java/org/apache/helix/participant/statemachine/ScheduledTaskStateModel.java > ca67d42 > > helix-core/src/main/java/org/apache/helix/participant/statemachine/ScheduledTaskStateModelFactory.java > a205910 > > helix-core/src/main/java/org/apache/helix/participant/statemachine/StateModelParser.java > eddeaa5 > helix-core/src/main/java/org/apache/helix/task/TaskRunner.java 66abba6 > helix-core/src/main/java/org/apache/helix/task/TaskStateModel.java a44a8cb > helix-core/src/main/java/org/apache/helix/task/TaskStateModelFactory.java > 2537747 > helix-core/src/main/java/org/apache/helix/tools/ClusterSetup.java 6599b33 > helix-core/src/test/java/org/apache/helix/DummyProcessThread.java f51aa1d > helix-core/src/test/java/org/apache/helix/Mocks.java 0303f12 > helix-core/src/test/java/org/apache/helix/TestHelixTaskExecutor.java > a3b16e5 > helix-core/src/test/java/org/apache/helix/TestHelixTaskHandler.java 43b4407 > helix-core/src/test/java/org/apache/helix/TestHelper.java 879e727 > > helix-core/src/test/java/org/apache/helix/integration/TestAddStateModelFactoryAfterConnect.java > 5f37845 > > helix-core/src/test/java/org/apache/helix/integration/TestBatchMessageWrapper.java > 6a6837a > > helix-core/src/test/java/org/apache/helix/integration/TestCorrectnessOnConnectivityLoss.java > abb2a7b > > helix-core/src/test/java/org/apache/helix/integration/TestErrorPartition.java > 19af9a7 > > helix-core/src/test/java/org/apache/helix/integration/TestHelixConnection.java > 3d02ae8 > > helix-core/src/test/java/org/apache/helix/integration/TestMessageThrottle2.java > 496a16f > > helix-core/src/test/java/org/apache/helix/integration/TestMessagingService.java > 08954e5 > > helix-core/src/test/java/org/apache/helix/integration/TestMultiClusterController.java > c2f9a5c > > helix-core/src/test/java/org/apache/helix/integration/TestNonOfflineInitState.java > 105633a > > helix-core/src/test/java/org/apache/helix/integration/TestPartitionLevelTransitionConstraint.java > 823a9ce > > helix-core/src/test/java/org/apache/helix/integration/TestPreferenceListAsQueue.java > 06a2b56 > > helix-core/src/test/java/org/apache/helix/integration/TestResetInstance.java > 5804744 > > helix-core/src/test/java/org/apache/helix/integration/TestResetPartitionState.java > 4855b3d > > helix-core/src/test/java/org/apache/helix/integration/TestResetResource.java > 7d28931 > > helix-core/src/test/java/org/apache/helix/integration/TestSchedulerMessage.java > 89af602 > > helix-core/src/test/java/org/apache/helix/integration/TestStateTransitionTimeout.java > c4304b0 > helix-core/src/test/java/org/apache/helix/integration/TestZkReconnect.java > 18f6fd7 > > helix-core/src/test/java/org/apache/helix/integration/manager/TestConsecutiveZkSessionExpiry.java > 99986ef > > helix-core/src/test/java/org/apache/helix/integration/manager/TestHelixMultiClusterController.java > 18234b5 > > helix-core/src/test/java/org/apache/helix/integration/manager/TestParticipantManager.java > 309ab18 > > helix-core/src/test/java/org/apache/helix/integration/manager/TestStateModelLeak.java > d6d7bab > > helix-core/src/test/java/org/apache/helix/manager/zk/MockMultiClusterController.java > 7f8b1a3 > helix-core/src/test/java/org/apache/helix/manager/zk/MockParticipant.java > f107d3d > > helix-core/src/test/java/org/apache/helix/manager/zk/TestDefaultControllerMsgHandlerFactory.java > 8b5b30c > > helix-core/src/test/java/org/apache/helix/messaging/TestAsyncCallbackSvc.java > da686fe > > helix-core/src/test/java/org/apache/helix/mock/participant/DummyProcess.java > 9880605 > > helix-core/src/test/java/org/apache/helix/mock/participant/MockBootstrapModelFactory.java > 177e7c4 > > helix-core/src/test/java/org/apache/helix/mock/participant/MockBootstrapStateModel.java > 79367db > > helix-core/src/test/java/org/apache/helix/mock/participant/MockMSModelFactory.java > 9325934 > > helix-core/src/test/java/org/apache/helix/mock/participant/MockMSStateModel.java > 78d9832 > > helix-core/src/test/java/org/apache/helix/mock/participant/MockSchemataModelFactory.java > 525e764 > > helix-core/src/test/java/org/apache/helix/mock/participant/MockSchemataStateModel.java > c3c1fa5 > helix-core/src/test/java/org/apache/helix/model/TestConstraint.java 6f78427 > > helix-core/src/test/java/org/apache/helix/participant/MockZKHelixManager.java > efa30da > > helix-core/src/test/java/org/apache/helix/participant/TestDistControllerStateModelFactory.java > 26d65f0 > > helix-core/src/test/java/org/apache/helix/participant/statemachine/TestStateModelParser.java > 7c128fb > > helix-examples/src/main/java/org/apache/helix/examples/BootstrapHandler.java > f0922f3 > > helix-examples/src/main/java/org/apache/helix/examples/BootstrapProcess.java > 2506c01 > > helix-examples/src/main/java/org/apache/helix/examples/DummyParticipant.java > c6ab3a4 > helix-examples/src/main/java/org/apache/helix/examples/ExampleProcess.java > 840a963 > > helix-examples/src/main/java/org/apache/helix/examples/LeaderStandbyStateModelFactory.java > 43ac5de > > helix-examples/src/main/java/org/apache/helix/examples/LogicalModelExample.java > 6075d22 > > helix-examples/src/main/java/org/apache/helix/examples/MasterSlaveStateModelFactory.java > 71d1412 > > helix-examples/src/main/java/org/apache/helix/examples/OnlineOfflineStateModelFactory.java > daf03a9 > helix-examples/src/main/java/org/apache/helix/examples/Quickstart.java > 2f3a677 > > helix-provisioning/src/main/java/org/apache/helix/provisioning/participant/StatelessServiceStateModel.java > f653de8 > > helix-provisioning/src/main/java/org/apache/helix/provisioning/participant/StatelessServiceStateModelFactory.java > 19c1488 > > recipes/distributed-lock-manager/src/main/java/org/apache/helix/lockmanager/Lock.java > 2ca3153 > > recipes/distributed-lock-manager/src/main/java/org/apache/helix/lockmanager/LockFactory.java > ab423f4 > > recipes/rabbitmq-consumer-group/src/main/java/org/apache/helix/recipes/rabbitmq/Consumer.java > 0b164b3 > > recipes/rabbitmq-consumer-group/src/main/java/org/apache/helix/recipes/rabbitmq/ConsumerStateModel.java > 24e4a40 > > recipes/rabbitmq-consumer-group/src/main/java/org/apache/helix/recipes/rabbitmq/ConsumerStateTransitionHandlerFactory.java > 98cce35 > > recipes/rsync-replicated-file-system/src/main/java/org/apache/helix/filestore/FileStoreStateModel.java > 6eaf808 > > recipes/rsync-replicated-file-system/src/main/java/org/apache/helix/filestore/FileStoreStateModelFactory.java > 7e1938c > > recipes/task-execution/src/main/java/org/apache/helix/taskexecution/TaskStateModel.java > 3c1cab4 > > recipes/task-execution/src/main/java/org/apache/helix/taskexecution/TaskStateModelFactory.java > 6948237 > > recipes/user-defined-rebalancer/src/main/java/org/apache/helix/userdefinedrebalancer/Lock.java > 308ae14 > > recipes/user-defined-rebalancer/src/main/java/org/apache/helix/userdefinedrebalancer/LockFactory.java > c607b1b > > Diff: https://reviews.apache.org/r/24332/diff/ > > > Testing > ------- > > > Thanks, > > Zhen Zhang > >
