glusterfs.git/xlators/cluster/dht/src, branch v3.3.0qa26

cluster/distribute: fix log level of few messages

2012-03-07T18:46:51+00:00

Change-Id: Ibdeeb705e75a94bb96a1ae259be32ddd2ca5fec8
Signed-off-by: Amar Tumballi 
BUG: 797715
Reviewed-on: http://review.gluster.com/2885
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/distribute: send proper 'dict' in readdirp()

2012-03-07T17:24:03+00:00

if we do the STACK_WIND from readdirp_cbk(), we were not sending
the right dictionary which made our linkfile determination a
failure.

Change-Id: Ie3f26748608f1a81123fe475d8a42bf2717475c3
Signed-off-by: Amar Tumballi 
BUG: 799887
Reviewed-on: http://review.gluster.com/2869
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda

fops/removexattr: prevent users from removing glusterfs xattrs

2012-03-05T17:17:47+00:00

* Each xlator prevents the user from removing xlator-specific
  xattrs like trusted.gfid by handling it in respective removexattr
  functions.

* For xlators which did not define remove and fremovexattr,
  the functions have been implemented with appropriate checks.

xlator         |     fops-added
_______________|__________________________
               |
1. stripe      |     removexattr and fremovexattr
2. quota       |     removexattr and fremovexattr

Change-Id: I98e22109717978134378bc75b2eca83fefb2abba
BUG: 783525
Signed-off-by: Rajesh Amaravathi 
Reviewed-on: http://review.gluster.com/2836
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Jeff Darcy

cluster/rebalance: Fix expression in check_free_space

2012-03-01T07:25:50+00:00

Migration should fail if available space in dst < src

The expression was evaluating otherwise

Change-Id: Ief64ad8797e405d0cdc787eef49bfbf4051dffac
BUG: 798864
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.com/2847
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

Introduce new extended attribute: node-uuid

2012-02-23T07:19:44+00:00

Request for trusted.glusterfs.node-uuid returns pathinfo
like string but containing the UUID of glusterd instead
of the backend path for the requested file. This info
is benificial for tasks like parallel rebalance that will
make use of the UUID for data locality.

Change-Id: I766a09cc4a5f63aebd11c73107924a1b29242dcf
BUG: 772610
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.com/2614
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda 
Reviewed-by: Vijay Bellur

mempool: adjustments in pool sizes

2012-02-22T12:23:48+00:00

* while creating 'rpc_clnt', the caller knows what would be the ideal
  load on it, so an extra argument to set some pool sizes

* while creating 'rpcsvc', the caller knows what would be the ideal
  load of it, so an extra argument to set request pool size

* cli memory footprint is reduced

Change-Id: Ie245216525b450e3373ef55b654b4cd30741347f
Signed-off-by: Amar Tumballi 
BUG: 765336
Reviewed-on: http://review.gluster.com/2784
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

core: utilize mempool for frame->local allocations

2012-02-21T10:42:09+00:00

in each translator, which uses 'frame->local', we are using
GF_CALLOC/GF_FREE, which would be costly considering the
number of allocation happening in a lifetime of 'fop'. It
would be good to utilize the mem pool framework for xlator's
local structures, so there is no allocation overhead.

Change-Id: Ida6e65039a24d9c219b380aa1c3559f36046dc94
Signed-off-by: Amar Tumballi 
BUG: 765336
Reviewed-on: http://review.gluster.com/2772
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cluster/distribute: send the right 'dict' for check_is_linkfile()

2012-02-21T10:41:42+00:00

earlier we were using 'check_is_linkfile_wo_dict()' for readdirp_cbk(),
which is no more required now.

Change-Id: I52295e08f8c494b7a081cfc684164a7f72562d0d
Signed-off-by: Amar Tumballi 
BUG: 794641
Reviewed-on: http://review.gluster.com/2763
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cluster/dht: Support for hardlink rebalance when decommissioning

2012-02-19T12:47:50+00:00

The support for hardlink rebalance is only available for decommissioning
of a node. this can be triggered in two ways
1. remove-brick start
2. if decommission node value is set in vol file, then a normal rebalance
   command

The way we handle it is-
if (nlink > 1)
do
       * if src file doesnt have linkto xattr
                * mark src's linkto to the dst
       * else
                * perform a link on the dst
                * do a look up
                * if nlinks = dst.nlinks
                        * migrate data
                * else
                        * continue crawling
done
Signed-off-by: shishir gowda 
Change-Id: If43b5524b872fd1413e9f7aa7f436cb244e30d8d
BUG: 763844
Reviewed-on: http://review.gluster.com/2737
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

cluster/dht: Handle get cached/hashed subvol failures gracefully

2012-02-19T12:46:26+00:00

Change-Id: I7a41c2876be04acd166b2004d9aa66af078d32ea
BUG: 790328
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.com/2757
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi