> /home/jenkins/root/workspace/rackspace-regression-2GB/xlators/cluster/dht/src/dht-common.c: In function ‘dht_lookup_everywhere_done’:
> /home/jenkins/root/workspace/rackspace-regression-2GB/xlators/cluster/dht/src/dht-common.c:1229: warning: implicit declaration of function ‘dht_fill_dict_to_avoid_unlink_of_migrating_file’
>
> Guessing these are significant. :)
00:08:09: harsha@sysrq:~/repos/glusterfs(master)$ git grep
dht_fill_dict_to_avoid_unlink_of_migrating_file
xlators/cluster/dht/src/dht-common.c:dht_fill_dict_to_avoid_unlink_of_migrating_file
(dict_t *dict) {
xlators/cluster/dht/src/dht-common.c: ret =
dht_fill_dict_to_avoid_unlink_of_migrating_file
xlators/cluster/dht/src/dht-common.c: ret =
dht_fill_dict_to_avoid_unlink_of_migrating_file
xlators/cluster/dht/src/dht-common.c: ret =
dht_fill_dict_to_avoid_unlink_of_migrating_file
xlators/cluster/dht/src/dht-common.h:dht_fill_dict_to_avoid_unlink_of_migrating_file
(dict_t *dict);
00:08:10: harsha@sysrq:~/repos/glusterfs(master)$
00:08:23: harsha@sysrq:~/repos/glusterfs(release-3.6)$ git grep
dht_fill_dict_to_avoid_unlink_of_migrating_file
xlators/cluster/dht/src/dht-common.c: ret =
dht_fill_dict_to_avoid_unlink_of_migrating_file
00:08:24: harsha@sysrq:~/repos/glusterfs(release-3.6)$
Interestingly not implemented or defined, part of this patch.
~~~~
commit 13a044ab4d643a39d8138ab33226162ef125dbd3
Author: Venkatesh Somyajulu <vsomyaju@xxxxxxxxxx>
Date: Thu Sep 4 14:08:18 2014 -0400
cluster/dht: Added keys in dht_lookup_everywhere_done
Case where both cached (C1) and hashed file are found,
but hash does not point to above cached node (C1), then
dont unlink if either fd-is-open on hashed or
linkto-xattr is not found.
Change-Id: I7ef49b88d2c88bf9d25d3aa7893714e6c0766c67
BUG: 1138385
Signed-off-by: Venkatesh Somyajulu <vsomyaju@xxxxxxxxxx>
Change-Id: I86d0a21d4c0501c45d837101ced4f96d6fedc5b9
Signed-off-by: Venkatesh Somyajulu <vsomyaju@xxxxxxxxxx>
Reviewed-on-master: http://review.gluster.org/8429
Tested-by: Gluster Build System <jenkins@xxxxxxxxxxxxxxxxx>
Reviewed-by: susant palai <spalai@xxxxxxxxxx>
Reviewed-by: Raghavendra G <rgowdapp@xxxxxxxxxx>
Reviewed-by: Vijay Bellur <vbellur@xxxxxxxxxx>
Reviewed-on: http://review.gluster.org/8607
Reviewed-by: Jeff Darcy <jdarcy@xxxxxxxxxx>
~~~
I would blame dumb GCC, clang would have never progressed this even
for a build.
--
Religious confuse piety with mere ritual, the virtuous confuse
regulation with outcomes
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://supercolony.gluster.org/mailman/listinfo/gluster-devel
