Maor Lipchuk has posted comments on this change.

Change subject: engine: Remove quota command
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveQuotaCommand.java
Line 19:     private static final long serialVersionUID = 8037593564997497667L;
I'm not sure if you meant the serialVersionUid or the QuotaDao

For QuotaDao, It is actually being used in the executeCommand.
But looking at it now, It might be more accurate to fetch the dao in the 
executeCommand scope instead in the general command scope.

I will move it to the executeCommand.

The serialVersionUID when this class will be serialized.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia252d597dbed227d8235107f758ef2c8a2de46dd
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to