Eli Mesika has posted comments on this change.

Change subject: gluster: DAO for volume snapshot scheduling
......................................................................


Patch Set 2:

(3 comments)

http://gerrit.ovirt.org/#/c/36486/2/packaging/dbscripts/gluster_volume_snapshot_sp.sql
File packaging/dbscripts/gluster_volume_snapshot_sp.sql:

Line 271: Create or replace FUNCTION 
GetGlusterVolumeSnapshotScheduleByVolumeId(v_volume_id UUID)
Line 272:     RETURNS SETOF gluster_volume_snapshot_schedules_view STABLE
Line 273:     AS $procedure$
Line 274: BEGIN
Line 275:       RETURN QUERY SELECT *
also here
Line 276:       FROM gluster_volume_snapshot_schedules_view
Line 277:       WHERE volume_id = v_volume_id;
Line 278: END; $procedure$
Line 279: LANGUAGE plpgsql;


Line 281: Create or replace FUNCTION 
DeleteGlusterVolumeSnapshotScheduleByVolumeId(v_volume_id UUID)
Line 282:     RETURNS VOID
Line 283:     AS $procedure$
Line 284: BEGIN
Line 285:       DELETE FROM gluster_volume_snapshot_schedules
also here
Line 286:       WHERE volume_id = v_volume_id;
Line 287: END; $procedure$
Line 288: LANGUAGE plpgsql;
Line 289: 


Line 306:           execution_time = v_execution_time,
Line 307:           days = v_days,
Line 308:           end_by = v_end_by,
Line 309:           _update_date = LOCALTIMESTAMP
Line 310:       WHERE volume_id = v_volume_id;
also here
Line 311: END; $procedure$


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie44116df49acc302bfc1b745c00535c1fb0174ad
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Ramesh N <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: Shubhendu Tripathi <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to