Moti Asayag has posted comments on this change.
Change subject: core: Introducing AddPOSIXStorageDomainCommand
......................................................................
Patch Set 5: (2 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddPOSIXStorageDomainCommand.java
Line 9: public class AddPOSIXStorageDomainCommand<T extends
StorageDomainManagementParameter> extends AddStorageDomainCommon<T> {
wouldn't you rather maintaining for the CamelCase in the class name ?
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java
Line 234: AddPOSIXStorageDomain(1009, ActionGroup.CREATE_STORAGE_DOMAIN),
same about the name.
--
To view, visit http://gerrit.ovirt.org/3624
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iec75d8b309b32eae5d02e58ac9d96ca27bbf3622
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
