[Bugs] [Bug 1369187] fix bug in protocol/client lookup callback

bugzilla at redhat.com bugzilla at redhat.com
Mon Aug 22 20:18:34 UTC 2016


https://bugzilla.redhat.com/show_bug.cgi?id=1369187



--- Comment #2 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: http://review.gluster.org/15237 committed in release-3.8 by Pranith
Kumar Karampuri (pkarampu at redhat.com) 
------
commit 72af868ba4a0ff7e143adaec78168aaccda473ef
Author: Anuradha Talur <atalur at redhat.com>
Date:   Tue Aug 9 21:09:11 2016 +0530

    protocol/client: Unserialize xdata even if lookup fails

    Problem:
    AFR relies on xdata returned by lookup to determine if
    there are any files that need healing. This info is further
    used to optimize readdirp.

    In case of lookups with negative return value, client
    xlator was sending NULL xdata. Due to absence of xdata,
    AFR conservatively assumes that there are files that
    need healing, which is incorrect.

    Solution:
    Even in case of unsuccessful lookups, send the xdata received
    by protocol client so that higher xlators can get the info
    that they rely on.

            >Change-Id: Id3a1023eb536180888eb2c0b39050000b76f7226
            >BUG: 1366284
            >Signed-off-by: Anuradha Talur <atalur at redhat.com>
            >Reviewed-on: http://review.gluster.org/15120
            >Smoke: Gluster Build System <jenkins at build.gluster.org>
            >Reviewed-by: Poornima G <pgurusid at redhat.com>
            >Tested-by: Poornima G <pgurusid at redhat.com>
            >CentOS-regression: Gluster Build System
<jenkins at build.gluster.org>
            >NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
            >Reviewed-by: Ashish Pandey <aspandey at redhat.com>
            >Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>
            >Signed-off-by: Anuradha Talur <atalur at redhat.com>

    Change-Id: Id3a1023eb536180888eb2c0b39050000b76f7226
    BUG: 1369187
    Signed-off-by: Anuradha Talur <atalur at redhat.com>
    Reviewed-on: http://review.gluster.org/15237
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Dfaem8I7LL&a=cc_unsubscribe


More information about the Bugs mailing list