Greg Padgett has posted comments on this change. Change subject: storage: Default to NFSv3 if version not specified ......................................................................
Patch Set 2: Ryan, good point regarding having to execute the code to get the version. It could in fact change at runtime though due to (dis-)connections, making the caching more complicated. That, combined with the fact that it's only retrieving an object to read the version rather than doing anything heavyweight, makes me think the benefits wouldn't outweigh the added complexity. -- To view, visit http://gerrit.ovirt.org/8241 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I83307ecd8262688f3d4ddb43b62a79924ecfc62c Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Greg Padgett <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Greg Padgett <[email protected]> Gerrit-Reviewer: Ryan Harper <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
