Change in ovirt-engine[master]: engine: Fix few eclipse warning

masayag at redhat.com masayag at redhat.com
Wed Nov 6 14:26:12 UTC 2013


Moti Asayag has submitted this change and it was merged.

Change subject: engine: Fix few eclipse warning
......................................................................


engine: Fix few eclipse warning

The AuditLogableBase doesn't implement Serializable, therefore
it doesn't required any serial id.

The serial warning can be removed from CommandBase as well.

Clear the type from CommandBase.validate() signature which hides
the T type defined as a parameter class type on the class signature.

Change-Id: I4c6e80252f0a9d907488be6ede45271c936f5de6
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogableBase.java
4 files changed, 4 insertions(+), 7 deletions(-)

Approvals:
  Moti Asayag: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4c6e80252f0a9d907488be6ede45271c936f5de6
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list