From e7eb1f1ac0e4bf197c2fa7af030583cc9c93a11c Mon Sep 17 00:00:00 2001
From: N Balachandran <nbalacha@redhat.com>
Date: Fri, 28 Apr 2017 11:32:48 +0530
Subject: [PATCH 408/426] cluster/dht Remove redundant logs in dht rmdir

Removing redundant logs were introduced in
https://review.gluster.org/#/c/17065/

> BUG: 1446227
> Signed-off-by: N Balachandran <nbalacha@redhat.com>
> Reviewed-on: https://review.gluster.org/17130
> Smoke: Gluster Build System <jenkins@build.gluster.org>
> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
> Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>

Change-Id: I0d6055488b51a13c91d2121e87f653cdb94888b0
BUG: 1445195
Signed-off-by: N Balachandran <nbalacha@redhat.com>
Reviewed-on: https://code.engineering.redhat.com/gerrit/104671
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
---
 xlators/cluster/dht/src/dht-common.c | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c
index f9a5729..81afabe 100644
--- a/xlators/cluster/dht/src/dht-common.c
+++ b/xlators/cluster/dht/src/dht-common.c
@@ -8259,14 +8259,16 @@ dht_rmdir_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
         prev  = cookie;
         src   = prev;
 
-gf_log ("dht", GF_LOG_INFO, "dht_rmdir_lookup_cbk %s", local->loc.path);
+        gf_msg_debug (this->name, 0, "dht_rmdir_lookup_cbk %s",
+                      local->loc.path);
+
         readdirp_frame = local->main_frame;
         readdirp_local = readdirp_frame->local;
 
         if (op_ret != 0) {
 
                 gf_msg (this->name, GF_LOG_WARNING, op_errno,
-                        DHT_MSG_NOT_LINK_FILE_ERROR,
+                        DHT_MSG_FILE_LOOKUP_FAILED,
                         "lookup failed for %s on %s  (type=0%o)",
                         local->loc.path, src->name, stbuf->ia_type);
                 goto err;
@@ -8475,7 +8477,7 @@ dht_rmdir_is_subvol_empty (call_frame_t *frame, xlator_t *this,
 
                 gf_uuid_unparse(lookup_local->loc.gfid, gfid);
 
-                gf_msg_debug (this->name, 0,
+                gf_msg_trace (this->name, 0,
                               "looking up %s on subvolume %s, gfid = %s",
                               lookup_local->loc.path, src->name, gfid);
 
@@ -8497,7 +8499,7 @@ dht_rmdir_is_subvol_empty (call_frame_t *frame, xlator_t *this,
                                 lookup_local->loc.path, gfid);
 
                         gf_msg_debug (this->name, 0,
-                                     "looking up %s on subvolume %s, gfid = %s",
+                                      "looking up %s on subvol %s, gfid = %s",
                                       lookup_local->loc.path, src->name, gfid);
 
                         STACK_WIND_COOKIE (lookup_frame, dht_rmdir_lookup_cbk,
@@ -8506,7 +8508,7 @@ dht_rmdir_is_subvol_empty (call_frame_t *frame, xlator_t *this,
                 } else {
                         gf_msg_debug (this->name, 0,
                                       "Looking up linkfile target %s on "
-                                      " subvolume %s, gfid = %s",
+                                      " subvol %s, gfid = %s",
                                       lookup_local->loc.path, subvol->name,
                                       gfid);
 
@@ -8639,9 +8641,6 @@ dht_rmdir_readdirp_do (call_frame_t *readdirp_frame, xlator_t *this)
                 return 0;
         }
 
-        gf_msg_debug ("this->name", 0, "Calling dht_rmdir_readdirp_do for %p",
-                      readdirp_frame);
-
         STACK_WIND_COOKIE (readdirp_frame, dht_rmdir_readdirp_cbk,
                            local->hashed_subvol,
                            local->hashed_subvol,
-- 
1.8.3.1