diff options
author | Amar Tumballi <amarts@redhat.com> | 2012-05-23 12:59:02 +0530 |
---|---|---|
committer | Anand Avati <avati@redhat.com> | 2012-05-23 00:43:19 -0700 |
commit | bb8a0664ef36809d8b8e75fcb973a2089e5d08a6 (patch) | |
tree | 5c7c2300a8d4637c783b4a84507172838eadbe8d /xlators/cluster/dht/src/dht-rebalance.c | |
parent | 2eb4a5b86596fbcbb1c0baa9f1b7de0843896dd9 (diff) |
distribute: use global synctask 'env' instead of local
creating a local synctask_env can lead to creating of many more
syncop threads than required. The current syncop logic can handle
the scale-up/scale-down of threads depending on the load. Hence,
its neater to use global synctask env.
Change-Id: Id46f963a0190c0154513317ae03323db155ac15a
Signed-off-by: Amar Tumballi <amarts@redhat.com>
BUG: 823774
Reviewed-on: http://review.gluster.com/3412
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'xlators/cluster/dht/src/dht-rebalance.c')
-rw-r--r-- | xlators/cluster/dht/src/dht-rebalance.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c index b07e5e9690b..fd07a1c7ec1 100644 --- a/xlators/cluster/dht/src/dht-rebalance.c +++ b/xlators/cluster/dht/src/dht-rebalance.c @@ -938,11 +938,8 @@ int dht_start_rebalance_task (xlator_t *this, call_frame_t *frame) { int ret = -1; - dht_conf_t *conf = NULL; - conf = this->private; - - ret = synctask_new (conf->env, rebalance_task, + ret = synctask_new (this->ctx->env, rebalance_task, rebalance_task_completion, frame, frame); return ret; |