ctdb-common: Mark VacuumLimit tunable as obsolete

Use of this tunable was dropped over 5 years ago in commit
16837bc309aa9a86fc21d7f59a8fce0b947428a3.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Autobuild-User(master): Amitay Isaacs <amitay@samba.org>
Autobuild-Date(master): Fri Oct  4 07:07:21 UTC 2019 on sn-devel-184

diff --git a/ctdb/common/tunable.c b/ctdb/common/tunable.c
index 8d475858c5f3dc499d21ed3ccf13ba523c44db2b..70412b7f979a72738bf7caa8b985b5e8bd345f7f 100644 (file)
--- a/ctdb/common/tunable.c
+++ b/ctdb/common/tunable.c
@@ -105,7 +105,7 @@ static struct {
                offsetof(struct ctdb_tunable_list, vacuum_max_run_time) },
        { "RepackLimit", 10*1000, false,
                offsetof(struct ctdb_tunable_list, repack_limit) },
-       { "VacuumLimit", 5*1000, false,
+       { "VacuumLimit", 5*1000, true,
                offsetof(struct ctdb_tunable_list, vacuum_limit) },
        { "VacuumFastPathCount", 60, false,
                offsetof(struct ctdb_tunable_list, vacuum_fast_path_count) },

diff --git a/ctdb/doc/ctdb-tunables.7.xml b/ctdb/doc/ctdb-tunables.7.xml
index 191bcaffe467455c03df38ffcd19b8d00fd850c7..289b1d7324befa5df9a21a123a832ab49ceee86d 100644 (file)
--- a/ctdb/doc/ctdb-tunables.7.xml
+++ b/ctdb/doc/ctdb-tunables.7.xml
@@ -574,10 +574,6 @@ MonitorInterval=20
         <varname>RepackLimit</varname>, then the database is repacked
         to get rid of the freelist records to avoid fragmentation.
       </para>
-      <para>
-        Databases are repacked only if both <varname>RepackLimit</varname>
-        and <varname>VacuumLimit</varname> are exceeded.
-      </para>
     </refsect2>
 
     <refsect2>
@@ -683,20 +679,6 @@ MonitorInterval=20
       </para>
     </refsect2>
 
-    <refsect2>
-      <title>VacuumLimit</title>
-      <para>Default: 5000</para>
-      <para>
-        During vacuuming, if the number of deleted records are more than
-        <varname>VacuumLimit</varname>, then databases are repacked to
-        avoid fragmentation.
-      </para>
-      <para>
-        Databases are repacked only if both <varname>RepackLimit</varname>
-        and <varname>VacuumLimit</varname> are exceeded.
-      </para>
-    </refsect2>
-
     <refsect2>
       <title>VacuumMaxRunTime</title>
       <para>Default: 120</para>

diff --git a/ctdb/doc/ctdb.1.xml b/ctdb/doc/ctdb.1.xml
index 157dfad9dc138d7d21c66d360aec21ba551e47d8..e355752d1dbf7eb93be74faf76275440d7991f40 100644 (file)
--- a/ctdb/doc/ctdb.1.xml
+++ b/ctdb/doc/ctdb.1.xml
@@ -809,7 +809,6 @@ RecoveryDropAllIPs      = 120
 VacuumInterval          = 10
 VacuumMaxRunTime        = 120
 RepackLimit             = 10000
-VacuumLimit             = 5000
 VacuumFastPathCount     = 60
 MaxQueueDropMsg         = 1000000
 AllowUnhealthyDBRead    = 0

diff --git a/ctdb/doc/examples/config_migrate.sh b/ctdb/doc/examples/config_migrate.sh
index e0d01e770576a3bcde73260c568119a4a173535c..8983105f1c9b1e15a9c277225635123d6de5d1a3 100755 (executable)
--- a/ctdb/doc/examples/config_migrate.sh
+++ b/ctdb/doc/examples/config_migrate.sh
@@ -323,7 +323,6 @@ TickleUpdateInterval
 TraverseTimeout
 VacuumFastPathCount
 VacuumInterval
-VacuumLimit
 VacuumMaxRunTime
 VerboseMemoryNames
 EOF
@@ -336,6 +335,7 @@ check_removed_tunable ()
 
        grep -Fiqx "$_tunable" <<EOF
 NoIPHostOnAllDisabled
+VacuumLimit
 EOF
 }
 

diff --git a/ctdb/tests/UNIT/tool/ctdb.listvars.001.sh b/ctdb/tests/UNIT/tool/ctdb.listvars.001.sh
index a304942af4371035f5adadd366385abd3f90ce09..88f0fa4daec5c4c758186827304ae2b8b858e2d9 100755 (executable)
--- a/ctdb/tests/UNIT/tool/ctdb.listvars.001.sh
+++ b/ctdb/tests/UNIT/tool/ctdb.listvars.001.sh
@@ -41,7 +41,6 @@ RecoveryDropAllIPs         = 120
 VacuumInterval             = 10
 VacuumMaxRunTime           = 120
 RepackLimit                = 10000
-VacuumLimit                = 5000
 VacuumFastPathCount        = 60
 MaxQueueDropMsg            = 1000000
 AllowUnhealthyDBRead       = 0