Laszlo Hornyak has posted comments on this change.

Change subject: engine: cluster policy db support
......................................................................


Patch Set 8: (1 inline comment)

....................................................
File 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/scheduling/ClusterPolicyDaoImpl.java
Line 213: 
Line 214:     /**
Line 215:      * Helper class for cluster's policy units. this class is hidden 
from users of this dao. mapped into cluster policy
Line 216:      * class.
Line 217:      */
this code is a bit hard to understand, however it works, it would be great to 
return here later.
Line 218:     private static class ClusterPolicyUnit {
Line 219:         Guid clusterPolicyId;
Line 220:         Guid policyUnitId;
Line 221:         boolean filterSelected;


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc34c5eba1d68e0650df38053c97e5138de3dfad
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Noam Slomianko <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to