MB-24055 [Ephemeral]: disable Bloom filter 69/77269/7
authorDave Rigby <daver@couchbase.com>
Mon, 24 Apr 2017 15:21:39 +0000 (16:21 +0100)
committerDave Rigby <daver@couchbase.com>
Tue, 25 Apr 2017 14:16:56 +0000 (14:16 +0000)
The Bloom filter is still enabled for Ephemeral buckets, however this
is a waste of memory; as it serves no purpose for Ephemeral buckets.

Disable it - as well as saving any maintenance cost, it also reduces
the RSS of an empty Ephemeral bucket from:

    57800 KB
to:

    44632 KB

Change-Id: I430ec84dbb7ff795141500aa76329b7ae5f5cd17
Reviewed-on: http://review.couchbase.org/77269
Reviewed-by: Manu Dhundi <manu@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>
src/ephemeral_bucket.cc

index 15f213f..2eaaf45 100644 (file)
@@ -197,6 +197,9 @@ void EphemeralBucket::reconfigureForEphemeral(Configuration& config) {
     // Disable access scanner - we never create it anyway, but set to
     // disabled as to not mislead the user via stats.
     config.setAccessScannerEnabled(false);
+    // Disable Bloom filter - it is currently no use for us (both
+    // alive+deleted keys are kept in HashTable).
+    config.setBfilterEnabled(false);
     // Disable warmup - it is not applicable to Ephemeral buckets.
     config.setWarmup(false);
     // Disable TAP - not supported for Ephemeral.