[Wasm] Adapt setup of operators using hash tables to match setup befo…

…re DSL refactoring.
mutable-org · May 25, 2023 · 85bc52b · 85bc52b
1 parent bf0e54b
commit 85bc52b
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/backend/WasmOperator.cpp b/src/backend/WasmOperator.cpp
@@ -591,8 +591,8 @@ void HashBasedGrouping::execute(const Match<HashBasedGrouping> &M, setup_t setup
     // TODO: determine setup
     using PROBING_STRATEGY = QuadraticProbing;
     constexpr bool USE_CHAINED_HASHING = false;
-    constexpr uint64_t AGGREGATES_SIZE_THRESHOLD_IN_BITS = 64;
-    constexpr double HIGH_WATERMARK = 0.8;
+    constexpr uint64_t AGGREGATES_SIZE_THRESHOLD_IN_BITS = std::numeric_limits<uint64_t>::infinity();
+    constexpr double HIGH_WATERMARK = 0.7;
 
     const auto num_keys = M.grouping.group_by().size();
 
@@ -2723,9 +2723,9 @@ void SimpleHashJoin<UniqueBuild, Predicated>::execute(const Match<SimpleHashJoin
 {
     // TODO: determine setup
     using PROBING_STRATEGY = QuadraticProbing;
-    constexpr bool USE_CHAINED_HASHING = true;
-    constexpr uint64_t PAYLOAD_SIZE_THRESHOLD_IN_BITS = 64;
-    constexpr double HIGH_WATERMARK = 1.5;
+    constexpr bool USE_CHAINED_HASHING = false;
+    constexpr uint64_t PAYLOAD_SIZE_THRESHOLD_IN_BITS = std::numeric_limits<uint64_t>::infinity();
+    constexpr double HIGH_WATERMARK = 0.7;
 
     const auto ht_schema = M.build.schema().drop_constants().deduplicate();
 
@@ -3129,8 +3129,8 @@ void HashBasedGroupJoin::execute(const Match<HashBasedGroupJoin> &M, setup_t set
     // TODO: determine setup
     using PROBING_STRATEGY = QuadraticProbing;
     constexpr bool USE_CHAINED_HASHING = false;
-    constexpr uint64_t AGGREGATES_SIZE_THRESHOLD_IN_BITS = 64;
-    constexpr double HIGH_WATERMARK = 0.8;
+    constexpr uint64_t AGGREGATES_SIZE_THRESHOLD_IN_BITS = std::numeric_limits<uint64_t>::infinity();
+    constexpr double HIGH_WATERMARK = 0.7;
 
     auto &C = Catalog::Get();
     const auto num_keys = M.grouping.group_by().size();