----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24332/ -----------------------------------------------------------
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-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/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/tools/ClusterSetup.java 6599b33 helix-core/src/test/java/org/apache/helix/DummyProcessThread.java f51aa1d 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/TestErrorPartition.java 19af9a7 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/MockMSModelFactory.java 9325934 helix-core/src/test/java/org/apache/helix/mock/participant/MockSchemataModelFactory.java 525e764 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-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/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 Diff: https://reviews.apache.org/r/24332/diff/ Testing ------- Thanks, Zhen Zhang
