Change in ovirt-engine[master]: core: Remove vm_last_up_time member from VmDynamic

amureini at redhat.com amureini at redhat.com
Sun Feb 24 06:40:47 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Remove vm_last_up_time member from VmDynamic
......................................................................


core: Remove vm_last_up_time member from VmDynamic

Removed the private member "vm_last_up_time" from the VmDynamic class.
This member was never updated (in fact, there are no methods to modify
it!), and never queried (no such methods exists either), and was only
used in the equals(Object) and hashCode() methods.
Since its value is always null, it's useless to continue holding it.

Change-Id: Iab5358b836c3cb7dd351bfeb25f1a197e534c1d1
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmDynamic.java
1 file changed, 1 insertion(+), 11 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Allon Mureinik: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iab5358b836c3cb7dd351bfeb25f1a197e534c1d1
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list