diff options
author | Krutika Dhananjay <kdhananj@redhat.com> | 2013-02-04 17:38:16 +0530 |
---|---|---|
committer | Anand Avati <avati@redhat.com> | 2013-02-04 19:23:00 -0800 |
commit | 42a3cc275c819ee0929c371a33779a51d32eda3f (patch) | |
tree | 386d8fa0274bbd14c10c01f95b7a284cb938d7aa | |
parent | 5165114025ca5d0ddea8ac6f04bc9d5527e762ef (diff) |
glusterd: remove extra call to glusterd_volume_compute_cksum()
In the commit phase of volume create, checksum on volinfo is
computed twice - once in the call to glusterd_store_volinfo() and once,
further down, in the same function glusterd_op_create_volume().
Change-Id: I36f9426943cd48937d4946b4b4ef09f19f31d888
BUG: 812356
Signed-off-by: Krutika Dhananjay <kdhananj@redhat.com>
Reviewed-on: http://review.gluster.org/4463
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
-rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-volume-ops.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-volume-ops.c b/xlators/mgmt/glusterd/src/glusterd-volume-ops.c index 246a7a6f1a8..3734bf7f82a 100644 --- a/xlators/mgmt/glusterd/src/glusterd-volume-ops.c +++ b/xlators/mgmt/glusterd/src/glusterd-volume-ops.c @@ -1613,12 +1613,6 @@ glusterd_op_create_volume (dict_t *dict, char **op_errstr) goto out; } - ret = glusterd_volume_compute_cksum (volinfo); - if (ret) { - *op_errstr = gf_strdup ("Failed to compute checksum of volume"); - goto out; - } - volinfo->rebal.defrag_status = 0; list_add_tail (&volinfo->vol_list, &priv->volumes); vol_added = _gf_true; |