Description of problem: I see that multiple functions (_remove/dict_remove_foreach_fn) are doing same thing. Similarly there are multiple iteration functions that could be written in terms of others reducing the number of such functions. i.e. dict_foreach, dict_foreach_fnmatch, dict_foreach_match. This bug tracks the changes that need to be done to refactor that code out. Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
REVIEW: http://review.gluster.org/9121 (dict: Write dict_foreach_* in form of dict_foreach_match) posted (#2) for review on master by Pranith Kumar Karampuri (pkarampu)
REVIEW: http://review.gluster.org/9121 (dict: Write dict_foreach_* in form of dict_foreach_match) posted (#3) for review on master by Pranith Kumar Karampuri (pkarampu)
COMMIT: http://review.gluster.org/9121 committed in master by Vijay Bellur (vbellur) ------ commit ff9ffab7b4e991de6191f10909c32a7a08f20ada Author: Pranith Kumar K <pkarampu> Date: Thu Nov 13 22:39:44 2014 +0530 dict: Write dict_foreach_* in form of dict_foreach_match Change-Id: Iaa3454f7f3b6516660b1976bea63e39ea7795f8f BUG: 1164051 Signed-off-by: Pranith Kumar K <pkarampu> Reviewed-on: http://review.gluster.org/9121 Reviewed-by: Jeff Darcy <jdarcy> Reviewed-by: Krishnan Parthasarathi <kparthas> Tested-by: Gluster Build System <jenkins.com>
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.0, please open a new bug report. glusterfs-3.7.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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939 [2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user