Description of problem: Refactor the dht lookup code in order to make it easier to maintain. Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
REVIEW: https://review.gluster.org/20246 (cluster/dht: refactor dht_lookup) posted (#1) for review on master by N Balachandran
REVIEW: https://review.gluster.org/20267 (cluster/dht: Minor code cleanup) posted (#1) for review on master by N Balachandran
COMMIT: https://review.gluster.org/20267 committed in master by "N Balachandran" <nbalacha> with a commit message- cluster/dht: Minor code cleanup Removed extra variable. Change-Id: If43c47f6630454aeadab357a36d061ec0b53cdb5 updates: bz#1590385 Signed-off-by: N Balachandran <nbalacha>
COMMIT: https://review.gluster.org/20246 committed in master by "Amar Tumballi" <amarts> with a commit message- cluster/dht: refactor dht_lookup The dht lookup code is getting difficult to maintain due to its size. Refactoring the code will make it easier to modify it in future. Change-Id: Ic7cb5bf4f018504dfaa7f0d48cf42ab0aa34abdd updates: bz#1590385 Signed-off-by: N Balachandran <nbalacha>
REVIEW: https://review.gluster.org/20622 (cluster/dht: refactor dht_lookup_cbk) posted (#1) for review on master by N Balachandran
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-5.0, please open a new bug report. glusterfs-5.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] https://lists.gluster.org/pipermail/announce/2018-October/000115.html [2] https://www.gluster.org/pipermail/gluster-users/
Reopening as this is an umbrella BZ for many more changes to the rebalance process.
REVIEW: https://review.gluster.org/21816 (cluster/dht: refactor dht_lookup_cbk) posted (#1) for review on master by N Balachandran
REVIEW: https://review.gluster.org/21816 (cluster/dht: refactor dht_lookup_cbk) posted (#7) for review on master by N Balachandran
Reopening this as there will be more changes.
REVIEW: https://review.gluster.org/22407 (cluster/dht: refactor dht lookup functions) posted (#1) for review on master by N Balachandran
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-6.0, please open a new bug report. glusterfs-6.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] https://lists.gluster.org/pipermail/announce/2019-March/000120.html [2] https://www.gluster.org/pipermail/gluster-users/
REVIEW: https://review.gluster.org/22407 (cluster/dht: refactor dht lookup functions) merged (#10) on master by N Balachandran
REVIEW: https://review.gluster.org/22542 (cluster/dht: Refactor dht lookup functions) posted (#1) for review on master by N Balachandran
REVIEW: https://review.gluster.org/22542 (cluster/dht: Refactor dht lookup functions) merged (#3) on master by Amar Tumballi
Marking this Modified as I am done with the changes for now.