Michael Pasternak has posted comments on this change.

Change subject: restapi: add memory attribute to snapshot
......................................................................


Patch Set 20: (1 inline comment)

....................................................
Commit Message
Line 3: AuthorDate: 2013-05-30 12:15:56 +0300
Line 4: Commit:     Arik Hadas <aha...@redhat.com>
Line 5: CommitDate: 2013-06-11 16:41:48 +0300
Line 6: 
Line 7: restapi: add memory attribute to snapshot
this patch description is not infiltrative enough in my view (had to dig to the 
feature page), please consider copy/paste short desr. from the wiki, also i'd 
change the subject accordingly
Line 8: 
Line 9: Add memory attribute of type boolean to the snapshot entity, which is
Line 10: set according to the memory volume attribute of the snapshot business
Line 11: entity: if the volume is not empty the attribute is true, or false


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id04e7944c4ee58cc05aafc95b6bfc451742b4eff
Gerrit-PatchSet: 20
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to