Yair Zaslavsky has posted comments on this change.
Change subject: core: Persist all async task place holders
......................................................................
Patch Set 2: (3 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java
Line 302: }
Line 303:
Line 304: private boolean actionAllowed = false;
Line 305:
Line 306: protected void checkCanDoAction() {
Ravi, please take care of it.
Line 307: actionAllowed = getReturnValue().getCanDoAction() ||
internalCanDoAction();
Line 308: }
Line 309:
Line 310: public VdcReturnValueBase executeAction() {
Line 1226: * This method is called before executeAction and after
checkCanDoAction
Line 1227: * to insert the async task place holders for the child commands.
Line 1228: */
Line 1229: protected void insertAsyncTaskPlaceHolders() {
Line 1230: TransactionSupport.executeInNewTransaction(new
TransactionMethod<Void>() {
Please check with Requires, and change if works.
In case someone by mistake will do cpy paste and run in transaction, at least
it will be in same transaction (code will look strange, but no harm will be
done when it comes to transaction).
Line 1231: @Override
Line 1232: public Void runInTransaction() {
Line 1233: Map<Guid, CommandBase<?>> childCommands =
buildChildCommands();
Line 1234: for (CommandBase<?> command :
childCommands.values()) {
Line 1259: * @param runAsInternal
Line 1260: * @param context
Line 1261: * @return
Line 1262: */
Line 1263: protected CommandBase<?> constructCommand(VdcActionType
actionType,
Ravi, I also don't support moving to backend, but Backend bean resides in bll.
Can you please elaborate on what you meant?
Line 1264: VdcActionParametersBase parameters,
Line 1265: boolean runAsInternal,
Line 1266: CommandContext context) {
Line 1267: ExecutionHandler.evaluateCorrelationId(parameters);
--
To view, visit http://gerrit.ovirt.org/15561
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I66628b98298ed1f500946889604ad906f0a68022
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
