Idan Shaby has posted comments on this change.

Change subject: core: WAD should be Ignored on File Domain Disks
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.ovirt.org/#/c/31687/4/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/SecureDeletionHandler.java
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/SecureDeletionHandler.java:

Line 6: import org.ovirt.engine.core.dal.dbbroker.DbFacade;
Line 7: 
Line 8: public class SecureDeletionHandler {
Line 9: 
Line 10:     private SecureDeletionHandler(){};
> The ";" at the end is redundant (read: wrong)
Done
Line 11: 
Line 12:     /**
Line 13:      * Since the file system is responsible for handling block 
allocation, there is no need
Line 14:      * for posting zeros on file domains. This method gets the 
parameters of a command that may


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0df2c07282994557e149db922cc1d3a166c5aa8f
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Idan Shaby <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Tal Nisan <[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