Enable touchValue by default

Author: igchor

cachelib/cachebench/cache/Cache.h

@@ -69,7 +69,7 @@ class Cache {
   explicit Cache(const CacheConfig& config,
                  ChainedItemMovingSync movingSync = {},
                  std::string cacheDir = "",
-                 bool touchValue = false);
+                 bool touchValue = true);
 
   ~Cache();
 
@@ -372,7 +372,7 @@ class Cache {
   std::unique_ptr<ValueTracker> valueTracker_;
 
   // read entire value on find.
-  bool touchValue_{false};
+  bool touchValue_{true};
 
   // reading of the nand bytes written for the benchmark if enabled.
   const uint64_t nandBytesBegin_{0};

cachelib/cachebench/util/Config.h

@@ -198,7 +198,7 @@ struct StressorConfig : public JSONConfig {
 
   // If enabled, each value will be read on find. This is useful for measuring
   // performance of value access.
-  bool touchValue{false};
+  bool touchValue{true};
 
   uint64_t numOps{0};     // operation per thread
   uint64_t numThreads{0}; // number of threads that will run