MB-48529: Remove CheckpointConfig members default value

Legacy from hardcoded test values, now meaningless. In the new code
they're all forced to be initialized by EPConfig values.

Change-Id: I7e015d82896afde7abf5b1b71876e72bc0cb521a
Reviewed-on: http://review.couchbase.org/c/kv_engine/+/164333
Tested-by: Build Bot <[email protected]>
Reviewed-by: Dave Rigby <[email protected]>

Author: paolococchi

engines/ep/src/checkpoint_config.h

@@ -74,23 +74,23 @@ class CheckpointConfig {
 
     // Period of a checkpoint in terms of time in seconds.
     // Dynamic in EPConfig
-    std::atomic<rel_time_t> checkpointPeriod{5};
+    std::atomic<rel_time_t> checkpointPeriod;
 
     // Number of max items allowed in each checkpoint.
     // Dynamic in EPConfig
-    std::atomic<size_t> checkpointMaxItems{1000};
+    std::atomic<size_t> checkpointMaxItems;
 
     // Number of max checkpoints allowed.
     // Dynamic in EPConfig
-    std::atomic<size_t> maxCheckpoints{2};
+    std::atomic<size_t> maxCheckpoints;
 
     // Flag indicating if a new checkpoint is created once the number of items
     // in the current checkpoint is greater than the max number allowed.
     // Dynamic in EPConfig
-    std::atomic<bool> itemNumBasedNewCheckpoint{true};
+    std::atomic<bool> itemNumBasedNewCheckpoint;
 
     // Flag indicating if persistence is enabled.
-    bool persistenceEnabled{true};
+    bool persistenceEnabled;
 
-    CheckpointRemoval checkpointRemovalMode = CheckpointRemoval::Eager;
+    CheckpointRemoval checkpointRemovalMode;
 };