From 16c57bb8c06b659c4e6c19f0df386ad1141fe89d Mon Sep 17 00:00:00 2001 From: Jialiang Tan Date: Mon, 28 Oct 2024 18:45:22 -0700 Subject: [PATCH] [native] Remove native session native_join_spill_partition_bits --- .../presto_cpp/main/SessionProperties.cpp | 12 ++---------- .../presto_cpp/main/SessionProperties.h | 7 ++----- 2 files changed, 4 insertions(+), 15 deletions(-) diff --git a/presto-native-execution/presto_cpp/main/SessionProperties.cpp b/presto-native-execution/presto_cpp/main/SessionProperties.cpp index cf63b5a75c395..880d8231b83cf 100644 --- a/presto-native-execution/presto_cpp/main/SessionProperties.cpp +++ b/presto-native-execution/presto_cpp/main/SessionProperties.cpp @@ -175,18 +175,10 @@ SessionProperties::SessionProperties() { QueryConfig::kRowNumberSpillEnabled, boolToString(c.rowNumberSpillEnabled())); - addSessionProperty( - kJoinSpillPartitionBits, - "Native Execution only. The number of bits (N) used to calculate the " - "spilling partition number for hash join and RowNumber: 2 ^ N", - INTEGER(), - false, - QueryConfig::kJoinSpillPartitionBits, - std::to_string(c.rowNumberSpillEnabled())); - addSessionProperty( kSpillerNumPartitionBits, - "none", + "The number of bits (N) used to calculate the spilling " + "partition number for hash join and RowNumber: 2 ^ N", TINYINT(), false, QueryConfig::kSpillNumPartitionBits, diff --git a/presto-native-execution/presto_cpp/main/SessionProperties.h b/presto-native-execution/presto_cpp/main/SessionProperties.h index c9a5db98d3326..b19d49f634483 100644 --- a/presto-native-execution/presto_cpp/main/SessionProperties.h +++ b/presto-native-execution/presto_cpp/main/SessionProperties.h @@ -131,11 +131,8 @@ class SessionProperties { static constexpr const char* kWriterSpillEnabled = "native_writer_spill_enabled"; - /// The number of bits (N) used to calculate the spilling - /// partition number for hash join and RowNumber: 2 ^ N - static constexpr const char* kJoinSpillPartitionBits = - "native_join_spiller_partition_bits"; - + /// The number of bits (N) used to calculate the spilling partition number for + /// hash join and RowNumber: 2 ^ N static constexpr const char* kSpillerNumPartitionBits = "native_spiller_num_partition_bits";