Gilad Chaplik has posted comments on this change.

Change subject: core: scheduling: increase NetworkPolicyUnit performance
......................................................................


Patch Set 2:

(6 comments)

Thanks Moti!

....................................................
File 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/InterfaceDao.java
Line 70: 
Line 71:     /**
Line 72:      * Retrieves all interfaces for the given Cluster id.
Line 73:      *
Line 74:      * @param id
Done
Line 75:      *            the cluster id
Line 76:      * @return the list of interfaces
Line 77:      */
Line 78:     List<VdsNetworkInterface> getAllInterfacesForCluster(Guid 
clusterId);


....................................................
File 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/InterfaceDaoDbFacadeImpl.java
Line 171:     public List<VdsNetworkInterface> getAllInterfacesForCluster(Guid 
clusterId) {
Line 172:         MapSqlParameterSource parameterSource = 
getCustomMapSqlParameterSource()
Line 173:                 .addValue("cluster_id", clusterId);
Line 174: 
Line 175:         return 
getCallsHandler().executeReadList("GetInterfacesByClusterId",
your suggestion.

Done
Line 176:                 vdsNetworkInterfaceRowMapper,
Line 177:                 parameterSource);
Line 178:     }
Line 179: 


....................................................
File packaging/dbscripts/create_views.sql
Line 866:       vds_interface_statistics.tx_drop, 
vds_interface_statistics.iface_status, vds_interface.type, 
vds_interface.gateway,
Line 867:       vds_interface.subnet, vds_interface.addr, vds_interface.speed, 
vds_interface.vlan_id, vds_interface.bond_type,
Line 868:       vds_interface.bond_name, vds_interface.is_bond, 
vds_interface.bond_opts, vds_interface.mac_addr,
Line 869:       vds_interface.network_name, vds_interface.name, 
vds_static.vds_id, vds_static.vds_name,  vds_interface.id,
Line 870:       vds_interface.boot_protocol, vds_interface.mtu as mtu, 
vds_interface.bridged, 1 AS is_vds, vds_static.vds_group_id
Done
Line 871:   FROM vds_interface_statistics
Line 872:   JOIN vds_interface ON vds_interface_statistics.id = vds_interface.id
Line 873:   JOIN vds_static ON vds_interface.vds_id = vds_static.vds_id;
Line 874: 


....................................................
File packaging/dbscripts/network_sp.sql
Line 357: 
Line 358: END; $procedure$
Line 359: LANGUAGE plpgsql;
Line 360: 
Line 361: Create or replace FUNCTION GetInterfacesByClusterId(v_cluster_id UUID)
Done
Line 362: RETURNS SETOF vds_interface_view STABLE
Line 363:    AS $procedure$
Line 364: BEGIN
Line 365:    RETURN QUERY SELECT *


Line 361: Create or replace FUNCTION GetInterfacesByClusterId(v_cluster_id UUID)
Line 362: RETURNS SETOF vds_interface_view STABLE
Line 363:    AS $procedure$
Line 364: BEGIN
Line 365:    RETURN QUERY SELECT *
Done
Line 366:    FROM vds_interface_view
Line 367:    WHERE vds_group_id = v_cluster_id;
Line 368: END; $procedure$
Line 369: LANGUAGE plpgsql;


Line 362: RETURNS SETOF vds_interface_view STABLE
Line 363:    AS $procedure$
Line 364: BEGIN
Line 365:    RETURN QUERY SELECT *
Line 366:    FROM vds_interface_view
:) only a spoon is missing! 
Done.
Line 367:    WHERE vds_group_id = v_cluster_id;
Line 368: END; $procedure$
Line 369: LANGUAGE plpgsql;
Line 370: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1b394b5d50abcbf58c63cb2d2af3867197ddb788
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Liran Zelkha <lzel...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to