Omer Frenkel has posted comments on this change. Change subject: engine: Add created_by field to vm_static ......................................................................
Patch Set 4: looks ok, but i think field name should be createByUserId also, we might want to save user name as well, as the user can be deleted from the db, and then the id will be pretty useless, no? -- To view, visit http://gerrit.ovirt.org/16035 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I465f9c49d05f49d42814cd970c3df1ed763a7fb5 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Martin Betak <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
