Yair Zaslavsky has posted comments on this change.
Change subject: Change storageServer to parse some params as strings
......................................................................
Patch Set 1:
Dan - I totally agree with you here.
Engine should support using a map (using Java terms here) of Map<String,Object> and
not Map<String,String> - however, this would currently require more changes at
engine side than this patch.
As we would like to fix the nfs options feature ASAP, I suggested this fix.
--
To view, visit
http://gerrit.ovirt.org/5062
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4d1716a6f66a108d60f39fad14d92625a692d0e3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>