diff options
| author | Prashanth Pai <ppai@redhat.com> | 2016-03-07 14:38:05 +0530 |
|---|---|---|
| committer | Thiago da Silva <thiago@redhat.com> | 2016-03-07 11:58:41 -0800 |
| commit | d7af577eb42e4c8bbdcadbb45a46d3a37c98193e (patch) | |
| tree | cbf5c201ad201d731606550f19bad2edc9d441d0 /gluster/swift/obj | |
| parent | ea4750a366123f78411d90082733642376dc6afc (diff) | |
Remove redundant syscalls in GET path
This change removes redundant fstat() and fgetxattr() performed in
the GET path when object added from file interface is being accessed
for the first time via the object interface.
This is a manual backport of this change:
https://review.openstack.org/#/c/246365/
Change-Id: I29f56cef80c94779145e2948ba14f35817d46e0c
Signed-off-by: Prashanth Pai <ppai@redhat.com>
Reviewed-on: http://review.gluster.org/13624
Reviewed-by: Thiago da Silva <thiago@redhat.com>
Tested-by: Thiago da Silva <thiago@redhat.com>
Diffstat (limited to 'gluster/swift/obj')
| -rw-r--r-- | gluster/swift/obj/diskfile.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gluster/swift/obj/diskfile.py b/gluster/swift/obj/diskfile.py index 21e6cee..b776d0f 100644 --- a/gluster/swift/obj/diskfile.py +++ b/gluster/swift/obj/diskfile.py @@ -611,8 +611,8 @@ class DiskFile(object): self._metadata = read_metadata(self._fd) if not validate_object(self._metadata, self._stat): - create_object_metadata(self._fd) - self._metadata = read_metadata(self._fd) + self._metadata = create_object_metadata(self._fd, self._stat, + self._metadata) assert self._metadata is not None self._filter_metadata() |