Maor Lipchuk has submitted this change and it was merged. Change subject: core:Remove multiple columns in group by to use view ...................................................................... core:Remove multiple columns in group by to use view The following patch removes group by which is used for disks. The group by is being done to use array for storage for disk, and it is done on three columns. Since the view use many columns each time a new column is added we need to add it to the group by, which cause the views to be large and ugly. The suggested fix is to use a view specifically for storage which use those three columns, and use join on it when ever it is used. Signed-off-by: Maor Lipchuk <mlipchuk@redhat.com> Change-Id: I273c423009f4718bc527a6155745fda757771303 --- M backend/manager/dbscripts/create_views.sql 1 file changed, 17 insertions(+), 17 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6924 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I273c423009f4718bc527a6155745fda757771303 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Ayal Baron <abaron@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com>