Remove visibility of CouchKVS::compactDBInternal

All current needs may be fulfilled via the public compactDB method

Change-Id: Icf8a7cd1572b870a6b53213b04bc3e4c5f7de544
Reviewed-on: http://review.couchbase.org/c/kv_engine/+/136286
Reviewed-by: Ben Huddleston <ben.huddleston@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

engines/ep/tests/mock/mock_couch_kvstore.h

@@ -64,7 +64,6 @@ class MockCouchKVStore : public CouchKVStore {
                        dbFileRevMap) {
     }
 
-    using CouchKVStore::compactDBInternal;
     using CouchKVStore::setMb40415RegressionHook;
 
     /**

engines/ep/tests/module_tests/couch-kvstore_test.cc

@@ -1561,12 +1561,12 @@ TEST_F(CouchstoreTest, testV2WriteRead) {
 // we wouldn't potentially get a database header without the _local/document
 TEST_F(CouchstoreTest, MB40415_regression_test) {
     CompactionConfig config;
-    compaction_ctx cctx(config, 0);
+    auto ctx = std::make_shared<compaction_ctx>(config, 0);
 
     // Verify that if we would "fail" the precommit hook for some reason
     // the entire compaction would fail...
     kvstore->setMb40415RegressionHook(true);
-    EXPECT_FALSE(kvstore->compactDBInternal(&cctx, {}));
+    EXPECT_FALSE(kvstore->compactDB(ctx));
 }
 
 class CouchKVStoreMetaData : public ::testing::Test {};