Bug 1378684
Summary: | Poor smallfile read performance on Arbiter volume compared to Replica 3 volume | |||
---|---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Ravishankar N <ravishankar> | |
Component: | arbiter | Assignee: | Ravishankar N <ravishankar> | |
Status: | CLOSED CURRENTRELEASE | QA Contact: | ||
Severity: | unspecified | Docs Contact: | ||
Priority: | unspecified | |||
Version: | mainline | CC: | bugs, mpillai, pkarampu, psuriset, ravishankar, rcyriac, rsussman, shberry | |
Target Milestone: | --- | |||
Target Release: | --- | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | glusterfs-3.10.0 | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | ||
Clone Of: | 1377193 | |||
: | 1379528 (view as bug list) | Environment: | ||
Last Closed: | 2017-03-06 17:27:39 UTC | Type: | Bug | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | 1377193, 1378867 | |||
Bug Blocks: | 1379528 |
Description
Ravishankar N
2016-09-23 05:34:16 UTC
REVIEW: http://review.gluster.org/15548 (afr: Ignore GF_CONTENT_KEY in metadata heal check) posted (#1) for review on master by Ravishankar N (ravishankar) REVIEW: http://review.gluster.org/15548 (afr: Ignore GF_CONTENT_KEY in metadata heal check) posted (#2) for review on master by Ravishankar N (ravishankar) REVIEW: http://review.gluster.org/15548 (afr: Ignore gluster internal (virtual) xattrs in metadata heal check) posted (#3) for review on master by Ravishankar N (ravishankar) REVIEW: http://review.gluster.org/15548 (afr: Ignore gluster internal (virtual) xattrs in metadata heal check) posted (#4) for review on master by Ravishankar N (ravishankar) COMMIT: http://review.gluster.org/15548 committed in master by Pranith Kumar Karampuri (pkarampu) ------ commit 5afc6aba906a21aee19c2f1baaa7d9eb762ae0ac Author: Ravishankar N <ravishankar> Date: Fri Sep 23 10:47:03 2016 +0530 afr: Ignore gluster internal (virtual) xattrs in metadata heal check Problem: In arbiter configuration, posix-xlator in the arbiter brick always sets the GF_CONTENT_KEY in the response dict with a value 0. If the file size on the data bricks is more than quick-read's max-file-size (64kb default), those bricks don't set the key. Because of this difference in the no. of dict elements, afr triggers metadata heal in lookup code path, in turn leading to extra lookups+inodelks. Fix: Changed afr dict comparison logic to ignore all virtual xattrs and the on-disk ones that we should not be healing. Also removed is_virtual_xattr() function. The original callers to this function (upcall) don't seem to need it anymore. Change-Id: I05730bdd39d8fb0b9a49a5fc9c0bb01f0d3bb308 BUG: 1378684 Signed-off-by: Ravishankar N <ravishankar> Reviewed-on: http://review.gluster.org/15548 NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> Smoke: Gluster Build System <jenkins.org> Reviewed-by: Pranith Kumar Karampuri <pkarampu> This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.10.0, please open a new bug report. glusterfs-3.10.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution. [1] http://lists.gluster.org/pipermail/gluster-users/2017-February/030119.html [2] https://www.gluster.org/pipermail/gluster-users/ |