Change in vdsm[master]: Replace unused getFileList() with more flexible getFileStats()

fsimonce at redhat.com fsimonce at redhat.com
Thu Oct 31 00:06:49 UTC 2013


Federico Simoncelli has posted comments on this change.

Change subject: Replace unused getFileList() with more flexible getFileStats()
......................................................................


Patch Set 6: Code-Review+1

I'm not thrilled by the caseSensitive argument. I'll think about it. The rest looks good.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I550827c7b4c7e11fe09e41745fcc9d91249c6c23
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list