From d7af577eb42e4c8bbdcadbb45a46d3a37c98193e Mon Sep 17 00:00:00 2001 From: Prashanth Pai Date: Mon, 7 Mar 2016 14:38:05 +0530 Subject: 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 Reviewed-on: http://review.gluster.org/13624 Reviewed-by: Thiago da Silva Tested-by: Thiago da Silva --- gluster/swift/obj/diskfile.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'gluster/swift/obj/diskfile.py') 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() -- cgit