Yaniv Bronhaim has posted comments on this change.

Change subject: core : Fix for HibernateVm command
......................................................................


Patch Set 1: (2 inline comments)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandAsyncTask.java
Line 167:     }
Line 168: 
Line 169:     private VdcActionType getEndActionType(AsyncTasks dbAsyncTask) {
Line 170:         VdcActionType commandType = 
dbAsyncTask.getActionParameters().getCommandType();
Line 171:         if (!VdcActionType.Unknown.equals(commandType)) {
shouldn't Unkown type refer to an error ? how is it possible to have cached 
Unknown type .. if it's possible just explain as part of the commit message 
please
Line 172:             return commandType;
Line 173:         }
Line 174:         return dbAsyncTask.getaction_type();
Line 175:     }


....................................................
Commit Message
Line 6: 
Line 7: core : Fix for HibernateVm command
Line 8: 
Line 9: HibernateVm command needed this fix in CommandAsyncTask for endAction
Line 10: to work
what does this fix do? why does it need it? please add more explanation ...
Line 11: 
Line 12: Change-Id: I529b7b4222e76f87e98b6807a89046a71fb8c938


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I529b7b4222e76f87e98b6807a89046a71fb8c938
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to