diff options
| author | Poornima G <pgurusid@redhat.com> | 2017-02-17 12:17:18 +0530 |
|---|---|---|
| committer | Shyamsundar Ranganathan <srangana@redhat.com> | 2017-02-17 11:56:00 -0500 |
| commit | 8039fcf2571ad7fd9841c8e5605cad9b55d8645c (patch) | |
| tree | c21fb14048d9f57e0cbbc3b7d1f74fa577b3103e | |
| parent | 324268b61a197389012304ee4223629965c0261c (diff) | |
readdir-ahead: Remove unnecessary logging
dict_get_int can return < 0 when key is not found is a valid case.
Hence no need to log.
> Reviewed-on: https://review.gluster.org/16654
> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
> Smoke: Gluster Build System <jenkins@build.gluster.org>
> Reviewed-by: Jeff Darcy <jdarcy@redhat.com>
Change-Id: If0795b0f178adbb94b10efc563506993f7411962
BUG: 1423429
Signed-off-by: Poornima G <pgurusid@redhat.com>
Reviewed-on: https://review.gluster.org/16659
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
| -rw-r--r-- | xlators/performance/readdir-ahead/src/readdir-ahead.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/xlators/performance/readdir-ahead/src/readdir-ahead.c b/xlators/performance/readdir-ahead/src/readdir-ahead.c index 17569c2a089..c6b167725a5 100644 --- a/xlators/performance/readdir-ahead/src/readdir-ahead.c +++ b/xlators/performance/readdir-ahead/src/readdir-ahead.c @@ -569,12 +569,6 @@ rda_opendir(call_frame_t *frame, xlator_t *this, loc_t *loc, fd_t *fd, local->xattrs = xdata_from_req; ret = dict_get_int32 (xdata, GF_READDIR_SKIP_DIRS, &local->skip_dir); - if (ret < 0) { - gf_msg (this->name, GF_LOG_ERROR, - 0, READDIR_AHEAD_MSG_DICT_OP_FAILED, - "Dict get of key:%s failed with :%d", - GF_READDIR_SKIP_DIRS, ret); - } frame->local = local; } |