----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17888/ -----------------------------------------------------------
(Updated Feb. 17, 2014, 11:45 p.m.) Review request for cloudstack, Alena Prokharchyk, daan Hoogland, and Hugo Trippaers. Changes ------- Just rebase the changes with all these last commit so that when the patch is finally pushed there are no conflicts. Repository: cloudstack-git Description ------- Dispatcher corrections, refactoring and tests. Corrects problems from previous attempts that were reverted by Alena. Most of the changes are the same, but this one is not creating conflicts of Map types for Aync Commands or for parameters as Lists or Maps. Diffs (updated) ----- api/src/org/apache/cloudstack/api/ApiConstants.java 3b9b19c api/src/org/apache/cloudstack/api/BaseCmd.java 0e83cee api/src/org/apache/cloudstack/api/BaseListCmd.java c1a4b4c api/src/org/apache/cloudstack/api/command/user/autoscale/CreateAutoScaleVmProfileCmd.java bee1b22 server/resources/META-INF/cloudstack/core/spring-server-core-misc-context.xml fd2f5fb server/src/com/cloud/api/ApiAsyncJobDispatcher.java 71ac616 server/src/com/cloud/api/ApiDispatcher.java ed95c72 server/src/com/cloud/api/ApiServer.java d715db6 server/src/com/cloud/api/ApiServlet.java 46f7eba server/src/com/cloud/api/dispatch/CommandCreationWorker.java PRE-CREATION server/src/com/cloud/api/dispatch/DispatchChain.java PRE-CREATION server/src/com/cloud/api/dispatch/DispatchChainFactory.java PRE-CREATION server/src/com/cloud/api/dispatch/DispatchTask.java PRE-CREATION server/src/com/cloud/api/dispatch/DispatchWorker.java PRE-CREATION server/src/com/cloud/api/dispatch/ParamGenericValidationWorker.java PRE-CREATION server/src/com/cloud/api/dispatch/ParamProcessWorker.java PRE-CREATION server/src/com/cloud/api/dispatch/ParamSemanticValidationWorker.java PRE-CREATION server/src/com/cloud/api/dispatch/ParamUnpackWorker.java PRE-CREATION server/src/com/cloud/network/as/AutoScaleManagerImpl.java 21c38ce server/src/com/cloud/storage/snapshot/SnapshotSchedulerImpl.java 3159059 server/test/com/cloud/api/ApiDispatcherTest.java 7314a57 server/test/com/cloud/api/dispatch/CommandCreationWorkerTest.java PRE-CREATION server/test/com/cloud/api/dispatch/DispatchChainFactoryTest.java PRE-CREATION server/test/com/cloud/api/dispatch/ParamGenericValidationWorkerTest.java PRE-CREATION server/test/com/cloud/api/dispatch/ParamProcessWorkerTest.java PRE-CREATION server/test/com/cloud/api/dispatch/ParamSemanticValidationWorkerTest.java PRE-CREATION Diff: https://reviews.apache.org/r/17888/diff/ Testing ------- Full build and test plus manually testing many features. Also including CreateTagsCommand that failed in previous commit. Thanks, Antonio Fornie
