Selvasundaram has posted comments on this change.

Change subject: engine: Get gluster hooks query
......................................................................


Patch Set 4: (3 inline comments)

....................................................
File backend/manager/dbscripts/gluster_volumes_sp.sql
Line 430: _update_date = LOCALTIMESTAMP
Line 431: WHERE id = v_volume_id;
Line 432: END; $procedure$
Line 433: LANGUAGE plpgsql;
Line 434: 
Done
Line 435: -- Gluster hooks
Line 436: Create or replace FUNCTION InsertGlusterHook(v_id UUID,
Line 437:                                 v_cluster_id UUID,
Line 438:                                 v_hook_name VARCHAR(1000),


....................................................
File backend/manager/dbscripts/upgrade/03_02_0310_add_gluster_hooks_table.sql
Line 21:        END IF;
Line 22: 
Line 23:        DROP INDEX IF EXISTS IDX_gluster_hooks_cluster_id;
Line 24:        CREATE INDEX IDX_gluster_hooks_cluster_id ON 
gluster_hooks(cluster_id);
Line 25:        DROP INDEX IF EXISTS IDX_gluster_hooks_name_unique;
Hook name is unique for a cluster. The table can have hooks info. for multiple 
clusters. The same hook name can exists in another cluster.
Line 26:        CREATE INDEX IDX_gluster_hooks_name_unique ON 
gluster_hooks(cluster_id, hook_name);
Line 27: 
Line 28: END; $procedure$
Line 29: LANGUAGE plpgsql;


....................................................
File 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterHookDaoTest.java
Line 90:         hookIds.add(HOOK_ID);
Line 91:         hookIds.add(HOOK_ID2);
Line 92:         dao.removeAll(hookIds);
Line 93:         List<GlusterHookEntity> hooks = dao.getByClusterId(CLUSTER_ID);
Line 94:         assertNotNull(hooks);
Done
Line 95:     }
Line 96: 
Line 97:     @Test
Line 98:     public void testUpdateGlusterHookStatus() {


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If8b2d261ead720fccea6884acbc806ec79e52b36
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Selvasundaram <sesub...@redhat.com>
Gerrit-Reviewer: Itamar Heim <ih...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Selvasundaram <sesub...@redhat.com>
Gerrit-Reviewer: Shireesh Anjal <san...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to