Ravi Nori has posted comments on this change.

Change subject: engine : Add quartz to handle AsycCommands
......................................................................


Patch Set 23:

(1 comment)

http://gerrit.ovirt.org/#/c/28160/23/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java:

Line 1304:                     public Void runInTransaction() {
Line 1305:                         buildChildCommandInfos();
Line 1306:                         for (Map.Entry<Guid, Pair<VdcActionType, 
VdcActionParametersBase>> entry : childCommandInfoMap.entrySet()) {
Line 1307:                             CommandBase<?> command =
Line 1308:                                     
BackendUtils.getBackendCommandObjectsHandler(log).createAction(entry.getValue().getFirst(),
> In the previous patch you were performing JNDI lookup to this bean somewher
I have made the changes in this patch
Line 1309:                                             
entry.getValue().getSecond());
Line 1310:                             command.insertAsyncTaskPlaceHolders();
Line 1311:                             childCommandsMap.put(entry.getKey(), 
command);
Line 1312:                         }


-- 
To view, visit http://gerrit.ovirt.org/28160
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I66b1e5945884aec412ba412e39266129004d7218
Gerrit-PatchSet: 23
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadg...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to