Updated Branches:
  refs/heads/vmsync bd0c239f6 -> 3327371f9

Add some comments to help collabrate with other developer


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/70728a74
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/70728a74
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/70728a74

Branch: refs/heads/vmsync
Commit: 70728a74713ab5fbe375bff83c34cc2eb2159a95
Parents: bd0c239
Author: Kelven Yang <kelv...@gmail.com>
Authored: Tue Jun 4 15:25:15 2013 -0700
Committer: Kelven Yang <kelv...@gmail.com>
Committed: Tue Jun 4 15:25:33 2013 -0700

----------------------------------------------------------------------
 .../framework/jobs/AsyncJobDispatcher.java         |    5 ++++
 .../cloudstack/framework/jobs/AsyncJobManager.java |   18 +++++++-------
 2 files changed, 14 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/70728a74/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobDispatcher.java
----------------------------------------------------------------------
diff --git 
a/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobDispatcher.java
 
b/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobDispatcher.java
index b46059c..5b0d15d 100644
--- 
a/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobDispatcher.java
+++ 
b/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobDispatcher.java
@@ -18,6 +18,11 @@ package org.apache.cloudstack.framework.jobs;
 
 import com.cloud.utils.component.Adapter;
 
+//
+// We extend it from Adapter interface for
+//     1)      getName()/setName()
+//     2)      Confirming to general adapter pattern used across CloudStack
+//
 public interface AsyncJobDispatcher extends Adapter {
        void runJob(AsyncJob job);
 }

http://git-wip-us.apache.org/repos/asf/cloudstack/blob/70728a74/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobManager.java
----------------------------------------------------------------------
diff --git 
a/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobManager.java 
b/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobManager.java
index 66c56e2..e577fb0 100644
--- 
a/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobManager.java
+++ 
b/framework/jobs/src/org/apache/cloudstack/framework/jobs/AsyncJobManager.java
@@ -1,4 +1,4 @@
-// Licensed to the Apache Software Foundation (ASF) under one
+// Licensed to the Apache Software Foundation (ASF) under ones
 // or more contributor license agreements.  See the NOTICE file
 // distributed with this work for additional information
 // regarding copyright ownership.  The ASF licenses this file
@@ -54,10 +54,10 @@ public interface AsyncJobManager extends Manager {
 
     /**
      * Used by upper level job to wait for completion of a down-level job 
(usually VmWork jobs)
-     * in synchronized way. Caller needs to use waitAndCheck() to check the 
completion status
+     * in synchronous way. Caller needs to use waitAndCheck() to check the 
completion status
      * of the down-level job
      * 
-     * Due to the amount of legacy code that relies on synchronized-call 
semantics, this form of joinJob
+     * Due to the amount of legacy code that relies on synchronous-call 
semantics, this form of joinJob
      * is used mostly
      * 
      * 
@@ -68,19 +68,19 @@ public interface AsyncJobManager extends Manager {
        
     /**
      * Used by upper level job to wait for completion of a down-level job 
(usually VmWork jobs)
-     * in asynchronized way, it will cause upper job to cease current 
execution, upper job will be
+     * in asynchronous way, it will cause upper job to cease current 
execution, upper job will be
      * rescheduled to execute periodically or on wakeup events detected from 
message bus
      * 
      * @param jobId upper job that is going to wait the completion of a 
down-level job
      * @param joinJobId down-level job
-     * @Param wakeupHandler    wakeup handler
-     * @Param wakeupDispatcher wakeup dispatcher
-     * @param wakeupTopcisOnMessageBus
+     * @Param wakeupHandler    wake-up handler
+     * @Param wakeupDispatcher wake-up dispatcher
+     * @param wakeupTopicsOnMessageBus
      * @param wakeupIntervalInMilliSeconds
      * @param timeoutInMilliSeconds
      */
     void joinJob(long jobId, long joinJobId, String wakeupHandler, String 
wakupDispatcher,
-               String[] wakeupTopcisOnMessageBus, long 
wakeupIntervalInMilliSeconds, long timeoutInMilliSeconds);
+               String[] wakeupTopicsOnMessageBus, long 
wakeupIntervalInMilliSeconds, long timeoutInMilliSeconds);
     
     /**
      * Dis-join two related jobs
@@ -107,7 +107,7 @@ public interface AsyncJobManager extends Manager {
     void syncAsyncJobExecution(AsyncJob job, String syncObjType, long 
syncObjId, long queueSizeLimit);
     
     /**
-     * This method will be deprecated after all code has been migrated to 
fully-asynchronized mode
+     * This method will be deprecated after all code has been migrated to 
fully-asynchronous mode
      * that uses async-feature of joinJob/disjoinJob
      * 
      * @param wakupTopicsOnMessageBus topic on message bus to wakeup the wait

Reply via email to