Eli Mesika has posted comments on this change.

Change subject: engine: watchdog - DB and logic changes
......................................................................


Patch Set 17: (1 inline comment)

....................................................
File backend/manager/dbscripts/create_views.sql
Line 769: 
Line 770: 
Line 771: 
Line 772: CREATE OR REPLACE VIEW vm_pools_full_view AS
Line 773:  SELECT vmp.vm_pool_id, vmp.vm_pool_name, vmp.vm_pool_description, 
vmp.vm_pool_type, vmp.parameters, vmp.prestarted_vms, vmp.vds_group_id, 
vmp.vds_group_name, vmp.max_assigned_vms_per_user, ( SELECT 
count(vm_pool_map.vm_pool_id) AS expr1
Why this was changed ? It seems not as a part of the original change
Line 774:            FROM vm_pools_view v1
Line 775:       LEFT JOIN vm_pool_map ON v1.vm_pool_id = vm_pool_map.vm_pool_id 
AND v1.vm_pool_id = vmp.vm_pool_id) AS assigned_vm_count, ( SELECT 
count(v2.vm_pool_id) AS expr1
Line 776:            FROM vm_pools v2
Line 777:       LEFT JOIN vm_pool_map vm_pool_map_1 ON v2.vm_pool_id = 
vm_pool_map_1.vm_pool_id AND v2.vm_pool_id = vmp.vm_pool_id


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I813a6f97e23008d15446285998a4e9b50b456040
Gerrit-PatchSet: 17
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <lhorn...@redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhorn...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to