Change in ovirt-engine[master]: engine: Host statuses - transaction

mkublin at redhat.com mkublin at redhat.com
Mon Mar 11 13:48:41 UTC 2013


Michael Kublin has submitted this change and it was merged.

Change subject: engine: Host statuses - transaction
......................................................................


engine: Host statuses - transaction

The following patch will move a transaction inside SetVdsStatusCommand, and transaction
will run only on updates of db and not whole SetVdsStatusCommand.
Also marked some commands by @NonTransactiveCommandAttribute, like MaintenanceVdsCommand and
SetNonOperationalVdsCommand because of only db operation is inside of SetVdsStatusCommand

Change-Id: I710a0474798036860f130c8b2bcc2e7a557714ea
Signed-off-by: Michael Kublin <mkublin at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ActivateVdsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MaintenanceVdsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestartVdsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SetNonOperationalVdsCommand.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/SetVdsStatusVDSCommand.java
5 files changed, 22 insertions(+), 28 deletions(-)

Approvals:
  Michael Kublin: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I710a0474798036860f130c8b2bcc2e7a557714ea
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list