Bug 1378305 - DHT: remove unused structure members
Summary: DHT: remove unused structure members
Alias: None
Product: GlusterFS
Classification: Community
Component: distribute
Version: mainline
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Nithya Balachandran
QA Contact:
Depends On:
TreeView+ depends on / blocked
Reported: 2016-09-22 04:51 UTC by Nithya Balachandran
Modified: 2017-03-06 17:26 UTC (History)
1 user (show)

Fixed In Version: glusterfs-3.10.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2017-03-06 17:26:53 UTC
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:

Attachments (Terms of Use)

Description Nithya Balachandran 2016-09-22 04:51:47 UTC
Description of problem:
Some structure members no longer seem to be used. These can be safely removed.
This helps by reducing unnecessary checks in the code.

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

How reproducible:

Steps to Reproduce:

Actual results:

Expected results:

Additional info:

Comment 1 Worker Ant 2016-09-22 05:14:01 UTC
REVIEW: http://review.gluster.org/15542 (cluster/dht: remove unnecessary struct member) posted (#1) for review on master by N Balachandran (nbalacha@redhat.com)

Comment 2 Worker Ant 2016-09-23 05:32:57 UTC
COMMIT: http://review.gluster.org/15542 committed in master by Raghavendra G (rgowdapp@redhat.com) 
commit 9d7d81e8058aa89fc24bcc211cfad3618720292d
Author: N Balachandran <nbalacha@redhat.com>
Date:   Thu Sep 22 10:38:21 2016 +0530

    cluster/dht: remove unnecessary struct member
    Removed a structure member that was not used
    from dht_local_t. Skipped some unnecessary checks
    in dht_filter_loc_subvol_key.
    Change-Id: I81740b6528e063fb9cf5817e05865ff4d77aa748
    BUG: 1378305
    Signed-off-by: N Balachandran <nbalacha@redhat.com>
    Reviewed-on: http://review.gluster.org/15542
    Smoke: Gluster Build System <jenkins@build.gluster.org>
    Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
    NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
    Reviewed-by: Raghavendra G <rgowdapp@redhat.com>

Comment 3 Shyamsundar 2017-03-06 17:26:53 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.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/

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