diff options
author | Peter Portante <peter.portante@redhat.com> | 2012-10-19 01:35:46 -0400 |
---|---|---|
committer | Anand Avati <avati@redhat.com> | 2012-10-25 15:04:35 -0700 |
commit | 8534cd56d8633c9468f337372d78c436263f36f6 (patch) | |
tree | 5271c9abd7de7e19fa66f3a907cf7b2506e94e50 /xlators/protocol/client/src | |
parent | 4f6aeb63d380f726b185bc25db1e50093b3119b3 (diff) |
Reduce the number of stat/fstat system calls made
This address BZ 868086, https://bugzilla.redhat.com/show_bug.cgi?id=868086
This refactoring consolidates the code a bit, and uses os.stat calls to get
all the information in one call when gathering the metadata for an object. The
five stat system calls (invoked via all the os.path.*() calls) have been
reduced to one.
We also added a unit test for the one new behavior where get_object_metadata()
will now throw an OSError exception if it has a problem stat'ing a file that
exists.
Change-Id: Iad5410c77938af68a47be757a3170abd201adeb0
BUG: 868086
Signed-off-by: Peter Portante <peter.portante@redhat.com>
Reviewed-on: http://review.gluster.org/4112
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Reviewed-by: Mohammed Junaid <junaid@redhat.com>
Tested-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'xlators/protocol/client/src')
0 files changed, 0 insertions, 0 deletions