Liron Ar has submitted this change and it was merged. Change subject: core: GetAllFromVmStaticByStoragePoolId - remove unneeded join ......................................................................
core: GetAllFromVmStaticByStoragePoolId - remove unneeded join remove unneeded join with vm_dynamic, vm_dynamic and vm_static should be added in the same transacation when adding a vm. I assume that this join was added to avoid querying templates, but it's unneeded as we already check the entity type in the query without performing that probably more expensive operation. Change-Id: I4fd372a5c4018d3c7bb7e38e1375fa198a14b572 Signed-off-by: Liron Aravot <[email protected]> --- M packaging/dbscripts/vms_sp.sql 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Liron Ar: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24716 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4fd372a5c4018d3c7bb7e38e1375fa198a14b572 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Ar <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Itamar Heim <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Sergey Gotliv <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
