Bug 1499150

Summary:	Improve performance with xattrop update.
Product:	[Community] GlusterFS	Reporter:	Sunil Kumar Acharya <sheggodu>
Component:	disperse	Assignee:	Sunil Kumar Acharya <sheggodu>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	medium
Version:	3.12	CC:	aspandey, bugs, jahernan
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-glusterfs-3.12.2	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1445663	Environment:
Last Closed:	2017-10-13 12:47:15 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:

Description Sunil Kumar Acharya 2017-10-06 09:15:52 UTC

+++ This bug was initially created as a clone of Bug #1445663 +++

Description of problem:
xattrop update on bricks is very slow.

--- Additional comment from Worker Ant on 2017-05-02 06:10:08 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#1) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-05-04 03:21:24 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#2) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-05-04 13:29:00 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#3) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-05-12 03:30:48 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#4) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-05-12 03:34:51 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#5) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-05-12 11:11:45 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#6) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-05-12 11:16:55 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#7) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-05-12 11:34:24 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#8) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-06-09 02:59:17 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#9) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-06-09 08:15:31 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#10) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-06-13 05:07:20 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#11) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-06-13 08:09:40 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#12) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-08-30 02:41:24 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#13) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-09-13 02:37:46 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#14) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-09-19 10:38:41 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#15) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-09-25 00:45:10 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#16) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-09-28 11:05:42 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#17) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-10-04 07:43:58 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#18) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-10-04 09:06:27 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#19) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-10-04 11:16:17 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#20) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-10-05 02:40:25 EDT ---

REVIEW: https://review.gluster.org/17151 (cluster/ec: Improve performance with xattrop update) posted (#21) for review on master by Sunil Kumar Acharya (sheggodu)

--- Additional comment from Worker Ant on 2017-10-05 21:42:10 EDT ---

COMMIT: https://review.gluster.org/17151 committed in master by Pranith Kumar Karampuri (pkarampu) 
------
commit 12cfc1983fa1521d6da94ad488145182cf97db2c
Author: Sunil Kumar Acharya <sheggodu>
Date:   Fri Apr 28 18:09:01 2017 +0530

    cluster/ec: Improve performance with xattrop update
    
    Existing EC code updates the xattr on the subvolume
    in a sequential pattern resulting in very poor performance.
    
    With this fix EC now updates the xattr on the subvolume
    in parallel which improves the xattr update performance.
    
    BUG: 1445663
    Change-Id: I3fc40d66db0b88875ca96a9fa01002ba386c0486
    Signed-off-by: Sunil Kumar Acharya <sheggodu>

Comment 1 Worker Ant 2017-10-06 09:27:40 UTC

REVIEW: https://review.gluster.org/18441 (cluster/ec: Improve performance with xattrop update) posted (#1) for review on release-3.12 by Sunil Kumar Acharya (sheggodu)

Comment 2 Worker Ant 2017-10-12 18:52:38 UTC

COMMIT: https://review.gluster.org/18441 committed in release-3.12 by jiffin tony Thottan (jthottan) 
------
commit b79588a90bb2fcb2f02b65b98e2ae265a926e507
Author: Sunil Kumar Acharya <sheggodu>
Date:   Fri Apr 28 18:09:01 2017 +0530

    cluster/ec: Improve performance with xattrop update
    
    Existing EC code updates the xattr on the subvolume
    in a sequential pattern resulting in very poor performance.
    
    With this fix EC now updates the xattr on the subvolume
    in parallel which improves the xattr update performance.
    
    >BUG: 1445663
    >Change-Id: I3fc40d66db0b88875ca96a9fa01002ba386c0486
    >Signed-off-by: Sunil Kumar Acharya <sheggodu>
    
    BUG: 1499150
    Change-Id: I3fc40d66db0b88875ca96a9fa01002ba386c0486
    Signed-off-by: Sunil Kumar Acharya <sheggodu>

Comment 3 Jiffin 2017-10-13 12:47:15 UTC

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-glusterfs-3.12.2, please open a new bug report.

glusterfs-glusterfs-3.12.2 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-October/032684.html
[2] https://www.gluster.org/pipermail/gluster-users/