Blob Blame History Raw
From 34a38c6f061259e34263f805e7424211df58b771 Mon Sep 17 00:00:00 2001
From: Poornima G <pgurusid@redhat.com>
Date: Fri, 12 May 2017 10:27:28 +0530
Subject: [PATCH 452/473] nl-cache: Remove the max limit for nl-cache-limit and
 nl-cache-timeout

The max limit is better unset when arbitrary. Otherwise in the future
if max has to be changed, it can break backward compatility.

>Reviewed-on: https://review.gluster.org/17261
>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: Jeff Darcy <jeff@pl.atyp.us>
>Signed-off-by: Poornima G <pgurusid@redhat.com>

Change-Id: I4337a3789a2d0d5cc8e2bf687a22536c97608461
BUG: 1450341
Signed-off-by: Poornima G <pgurusid@redhat.com>
Reviewed-on: https://code.engineering.redhat.com/gerrit/106410
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
---
 xlators/performance/nl-cache/src/nl-cache.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/xlators/performance/nl-cache/src/nl-cache.c b/xlators/performance/nl-cache/src/nl-cache.c
index 675b5de..26c09d7 100644
--- a/xlators/performance/nl-cache/src/nl-cache.c
+++ b/xlators/performance/nl-cache/src/nl-cache.c
@@ -762,7 +762,6 @@ struct volume_options options[] = {
         { .key = {"nl-cache-limit"},
           .type = GF_OPTION_TYPE_SIZET,
           .min = 0,
-          .max = 100 * GF_UNIT_MB,
           .default_value = "131072",
           .description = "the value over which caching will be disabled for"
                          "a while and the cache is cleared based on LRU",
@@ -770,7 +769,6 @@ struct volume_options options[] = {
         { .key = {"nl-cache-timeout"},
           .type = GF_OPTION_TYPE_TIME,
           .min = 0,
-          .max = 1 * GF_DAY_IN_SECONDS,
           .default_value = "60",
           .description = "Time period after which cache has to be refreshed",
         },
-- 
1.8.3.1