Change in ovirt-engine[master]: core: Removed un needed run of code in Suppres mode

yzaslavs at redhat.com yzaslavs at redhat.com
Thu Sep 27 10:02:32 UTC 2012


Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: Removed un needed run of code in Suppres mode
......................................................................


core: Removed un needed run of code in Suppres mode

The following patch will remove un needed run of code in Suppress mode.
The following code is already run with out scope of any transaction, no
need explicity to remove it.
Motivation: less code - better redability, less look ups - better performance,
less anonymous class - better memory.

Change-Id: I87050e78abf0e468ebae5348f6e01c9ce46c7581
Bug-Url: NA
Signed-off-by: Michael Kublin <mkublin at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
1 file changed, 13 insertions(+), 28 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I87050e78abf0e468ebae5348f6e01c9ce46c7581
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list