Bug 1335821 - Revert "features/shard: Make o-direct writes work with sharding: http://review.gluster.org/#/c/13846/"
Summary: Revert "features/shard: Make o-direct writes work with sharding: http://revie...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: sharding
Version: 3.7.11
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Krutika Dhananjay
QA Contact: bugs@gluster.org
URL:
Whiteboard:
Depends On: 1335818 1335822
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-05-13 09:59 UTC by Krutika Dhananjay
Modified: 2016-06-28 12:17 UTC (History)
1 user (show)

Fixed In Version: glusterfs-3.7.12
Doc Type: Bug Fix
Doc Text:
Clone Of: 1335818
Environment:
Last Closed: 2016-06-28 12:17:59 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Krutika Dhananjay 2016-05-13 09:59:04 UTC
+++ This bug was initially created as a clone of Bug #1335818 +++

Description of problem:
Revert this patch for two reasons:

1) It introduces high fop latencies
2) Even with the patch, there is no true odirect behavior since the workaround in the patch doesn't reduce the caching done in kernel's page cache etc.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Vijay Bellur on 2016-05-13 05:54:26 EDT ---

REVIEW: http://review.gluster.org/14328 (Revert "features/shard: Make o-direct writes work with sharding") posted (#1) for review on master by Krutika Dhananjay (kdhananj)

Comment 1 Vijay Bellur 2016-05-13 10:01:36 UTC
REVIEW: http://review.gluster.org/14329 (Revert "features/shard: Make o-direct writes work with sharding") posted (#1) for review on release-3.7 by Krutika Dhananjay (kdhananj)

Comment 2 Vijay Bellur 2016-05-17 11:01:48 UTC
COMMIT: http://review.gluster.org/14329 committed in release-3.7 by Pranith Kumar Karampuri (pkarampu) 
------
commit 34056a0740d87e0b56e40fbf53b46443b0c9c444
Author: Krutika Dhananjay <kdhananj>
Date:   Fri May 13 15:18:22 2016 +0530

    Revert "features/shard: Make o-direct writes work with sharding"
    
            Backport of: http://review.gluster.org/#/c/14328/
    
    This reverts commit c272c71391cea9db817f4e7e38cfc25a7cff8bd5.
    
    This is for two reasons:
    1) It introduces high fop latencies
    2) Even with the patch, there is no true odirect behavior since the
       workaround in the patch doesn't reduce the caching done in kernel's
       page cache as far as writes on anon fds associated with individual
       shards is concerned.
    
    Change-Id: Ia39f8f30a46a7559eafbb31658aed8c8985be593
    BUG: 1335821
    Signed-off-by: Krutika Dhananjay <kdhananj>
    Reviewed-on: http://review.gluster.org/14329
    Smoke: Gluster Build System <jenkins.com>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.com>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>

Comment 3 Kaushal 2016-06-28 12:17:59 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-3.7.12, please open a new bug report.

glusterfs-3.7.12 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] https://www.gluster.org/pipermail/gluster-devel/2016-June/049918.html
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user


Note You need to log in before you can comment on or make changes to this bug.