Yair Zaslavsky has posted comments on this change.

Change subject: extapi: Changing PrincipalRecord to contain principal field
......................................................................


Patch Set 4:

Please advise,
I don't understand - are we missing here change to fetch principal or not? the 
comment you suggested me to put on the principal field is confusing (based on 
what you wrote on the next patch that search should be based on principal in 
some cases).

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6d2b532f56ef5b3a38af79ef869c5a5c8bf526bf
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to