Idan Shaby has posted comments on this change.

Change subject: webadmin: Configurable Default for Wipe After Delete per 
Storage Domain
......................................................................


Patch Set 5:

(1 comment)

http://gerrit.ovirt.org/#/c/36526/5/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetStorageDomainWipeAfterDeleteQuery.java
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetStorageDomainWipeAfterDeleteQuery.java:

Line 9:     public GetStorageDomainWipeAfterDeleteQuery(P parameters) {
Line 10:         super(parameters);
Line 11:     }
Line 12: 
Line 13:     @Override protected void executeQueryCommand() {
> style - move the '@Override' annotation to a separate line above
Done
Line 14:         if (getParameters().getStorageType() != null) {
Line 15:             getQueryReturnValue().setReturnValue(
Line 16:                     
WipeAfterDeleteUtils.getDefaultWipeAfterDeleteFlag(getParameters().getStorageType()));
Line 17:         }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id0bf6a95eb4b33867de86e99f8cb59eec0d1cdd4
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Amit Aviram <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Idan Shaby <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to