Remove test for Jumphash in Memento

Author: slashdotted

memento/mementoengine.h

@@ -38,12 +38,6 @@ class MementoEngine final {
    */
   uint32_t getBucket(std::string_view key) const noexcept {
     const auto hash{XXH64(key.data(), key.size(), 0)};
-
-    /* If there are no removed nodes, just perform Jump */
-    if (m_lastRemoved == m_bArraySize) {
-      return JumpConsistentHash(hash, m_bArraySize);
-    }
-
     /*
      * We invoke JumpHash to get a bucket
      * in the range [0,bArraySize-1].
@@ -96,12 +90,6 @@ class MementoEngine final {
    */
   uint32_t getBucketCRC32c(uint64_t key, uint64_t seed) const noexcept {
     const auto hash = crc32c_sse42_u64(key, seed);
-
-    /* If there are no removed nodes, just perform Jump */
-    if (m_lastRemoved == m_bArraySize) {
-      return JumpConsistentHash(hash, m_bArraySize);
-    }
-
     /*
      * We invoke JumpHash to get a bucket
      * in the range [0,bArraySize-1].

speed_test.cpp

@@ -152,13 +152,13 @@ int bench(const std::string_view name, const std::string &filename,
   print_memory_stats("AfterRemovals");
 #endif
 
-  int64_t sum{0};
+  volatile int64_t bucket{0};
   auto start{clock()};
   for (uint32_t i = 0; i < num_keys; ++i) {
 #ifdef USE_PCG32
-    sum += engine.getBucketCRC32c(rng(), rng());
+    bucket = engine.getBucketCRC32c(rng(), rng());
 #else
-    sum += engine.getBucketCRC32c(rand(), rand());
+    bucket = engine.getBucketCRC32c(rand(), rand());
 #endif
   }
   auto end{clock()};
@@ -170,7 +170,7 @@ int bench(const std::string_view name, const std::string &filename,
   auto elapsed{static_cast<double>(end - start) / CLOCKS_PER_SEC};
 #ifdef USE_HEAPSTATS
   auto maxheap{maximum};
-  fmt::println("{} Elapsed time is {} seconds, maximum heap allocated memory is {} bytes, sizeof({}) is {}, sum is {}", name, elapsed, maxheap, name, sizeof(Algorithm), sum);
+  fmt::println("{} Elapsed time is {} seconds, maximum heap allocated memory is {} bytes, sizeof({}) is {}", name, elapsed, maxheap, name, sizeof(Algorithm));
   results_file << name << ":\tAnchor\t" << anchor_set << "\tWorking\t"
                << working_set << "\tRemovals\t" << num_removals << "\tRate\t"
                << norm_keys_rate / elapsed << "\tMaxHeap\t" << maxheap << "\tAlgoSizeof\t" << sizeof(Algorithm)<< "\n";