Lowered estimation max error

Author: Enmk

ut/ColumnString_ut.cpp

@@ -47,7 +47,7 @@ size_t EstimateColumnStringMemoryUsage(
             ? COLUMN_STRING_DEFAULT_BLOCK_SIZE % static_cast<size_t>(static_cast<size_t>(item_estimated_size) * value_to_estimation_average_size_ratio)
             : COLUMN_STRING_DEFAULT_BLOCK_SIZE / 10);
 
-    const auto max_estimation_error_factor = item_estimated_size == ColumnString::NO_PREALLOCATE ? 2.5 : 2;
+    const auto max_estimation_error_factor = item_estimated_size == ColumnString::NO_PREALLOCATE ? 2 : 1.2;
 
     return (number_of_items * sizeof(std::string_view)
         + estimated_total_item_size

ut/utils_ut.cpp

@@ -158,15 +158,16 @@ TEST_F(OutputTest, PrettyPrintByteSize)
                 {3.25, "3.25"},
                 {13.75, "13.75"},
                 {135.5, "135.5"},
-                {135.125, "135.125"},
+                {135.125, "135.12"},
                 {10, "10"},
                 {100, "100"},
                 {1000, "1000"},
              })
         {
             const auto bytes_value = static_cast<size_t>(base * value);
             const auto expected_str = std::string(value_str) + " " + base_name;
-            EXPECT_EQ(expected_str, ToString(PrettyPrintByteSize{bytes_value}))
+
+            EXPECT_EQ(expected_str, ToString(PrettyPrintByteSize{bytes_value, 2}))
                 << "\n\tbase:      " << base
                 << "\n\tbase_name: " << base_name
                 << "\n\tvalue:     " << value