rename to geotoh3_argument_order and use enum values

Author: bharatnc

CHANGELOG.md

@@ -18,7 +18,7 @@
 ### <a id="255"></a> ClickHouse release 25.5, 2025-05-22
 
 #### Backward Incompatible Change
-* Function `geoToH3` now accepts the input in the order (lat, lon, res) (which is common for other geometric functions). Users who wish to retain the previous result order (lon, lat, res) can set setting `geotoh3_lon_lat_argument_order = true`. [#78852](https://github.com/ClickHouse/ClickHouse/pull/78852) ([Pratima Patel](https://github.com/pratimapatel2008)).
+* Function `geoToH3` now accepts the input in the order (lat, lon, res) (which is common for other geometric functions). Users who wish to retain the previous result order (lon, lat, res) can set setting `geotoh3_argument_order = 'lon_lat'`. [#78852](https://github.com/ClickHouse/ClickHouse/pull/78852) ([Pratima Patel](https://github.com/pratimapatel2008)).
 * Add a filesystem cache setting `allow_dynamic_cache_resize`, by default `false`, to allow dynamic resize of filesystem cache. Why: in certain environments (ClickHouse Cloud) all the scaling events happen through the restart of the process and we would love this feature to be explicitly disabled to have more control over the behaviour + as a safety measure. This PR is marked as backward incompatible, because in older versions dynamic cache resize worked by default without special setting. [#79148](https://github.com/ClickHouse/ClickHouse/pull/79148) ([Kseniia Sumarokova](https://github.com/kssenii)).
 * Removed support for legacy index types `annoy` and `usearch`. Both have been stubs for a long time, i.e. every attempt to use the legacy indexes returned an error anyways. If you still have `annoy` and `usearch` indexes, please drop them. [#79802](https://github.com/ClickHouse/ClickHouse/pull/79802) ([Robert Schulze](https://github.com/rschu1ze)).
 * Remove `format_alter_commands_with_parentheses` server setting. The setting was introduced and disabled by default in 24.2. It was enabled by default in 25.2. As there are no LTS versions that don't support the new format, we can remove the setting. [#79970](https://github.com/ClickHouse/ClickHouse/pull/79970) ([János Benjamin Antal](https://github.com/antaljanosbenjamin)).

docs/en/sql-reference/functions/geo/h3.md

@@ -210,7 +210,7 @@ geoToH3(lat, lon, resolution)
 - Hexagon index number. [UInt64](../../data-types/int-uint.md).
 - 0 in case of error. [UInt64](../../data-types/int-uint.md).
 
-Note: In ClickHouse v25.4 or older, `geoToH3()` takes values in order `(lon, lat)`. As per ClickHouse v25.5, the input values are in order `(lat, lon)`. The previous behaviour can be restored using setting `geotoh3_lon_lat_argument_order = true`.
+Note: In ClickHouse v25.4 or older, `geoToH3()` takes values in order `(lon, lat)`. As per ClickHouse v25.5, the input values are in order `(lat, lon)`. The previous behaviour can be restored using setting `geotoh3_argument_order = 'lon_lat'`.
 
 **Example**
 

src/Core/Settings.cpp

@@ -3651,8 +3651,8 @@ If enabled, functions 'least' and 'greatest' return NULL if one of their argumen
     DECLARE(Bool, h3togeo_lon_lat_result_order, false, R"(
 Function 'h3ToGeo' returns (lon, lat) if true, otherwise (lat, lon).
 )", 0) \
-    DECLARE(Bool, geotoh3_lon_lat_argument_order, false, R"(
-Function 'geoToH3' accepts (lon, lat) if true, otherwise (lat, lon).
+    DECLARE(String, geotoh3_argument_order, "lat_lon", R"(
+Function 'geoToH3' accepts (lon, lat) if set to 'lon_lat' and (lat, lon) if set to 'lat_lon'.
 )", 0) \
     DECLARE(UInt64, max_partitions_per_insert_block, 100, R"(
 Limits the maximum number of partitions in a single inserted block

src/Core/SettingsChangesHistory.cpp

@@ -74,7 +74,7 @@ const VersionToSettingsChangesMap & getSettingsChangesHistory()
         addSettingsChanges(settings_changes_history, "25.5",
         {
             /// Release closed. Please use 25.6
-            {"geotoh3_lon_lat_argument_order", true, false, "A new setting for legacy behaviour to set lon and lat argument order"},
+            {"geotoh3_argument_order", "", "", "A new setting for legacy behaviour to set lon and lat argument order"},
             {"secondary_indices_enable_bulk_filtering", false, true, "A new algorithm for filtering by data skipping indices"},
             {"implicit_table_at_top_level", "", "", "A new setting, used in clickhouse-local"},
             {"use_skip_indexes_if_final_exact_mode", 0, 0, "This setting was introduced to help FINAL query return correct results with skip indexes"},

src/Functions/geoToH3.cpp

@@ -20,7 +20,7 @@ namespace DB
 {
 namespace Setting
 {
-    extern const SettingsBool geotoh3_lon_lat_argument_order;
+    extern const SettingsString geotoh3_argument_order;
 }
 namespace ErrorCodes
 {
@@ -37,15 +37,21 @@ namespace
 /// and returns h3 index of this point
 class FunctionGeoToH3 : public IFunction
 {
-    const bool geotoh3_lon_lat_argument_order;
+    const String geotoh3_argument_order;
 public:
     static constexpr auto name = "geoToH3";
 
     static FunctionPtr create(ContextPtr context) { return std::make_shared<FunctionGeoToH3>(context); }
 
     explicit FunctionGeoToH3(ContextPtr context)
-    : geotoh3_lon_lat_argument_order(context->getSettingsRef()[Setting::geotoh3_lon_lat_argument_order])
+    : geotoh3_argument_order(context->getSettingsRef()[Setting::geotoh3_argument_order])
     {
+        // Validate that the setting value is either "lat_lon" or "lon_lat".
+        if (geotoh3_argument_order != "lat_lon" || geotoh3_argument_order != "lon_lat")
+            throw Exception(
+                ErrorCodes::ILLEGAL_TYPE_OF_ARGUMENT,
+                "Invalid value for 'geotoh3_argument_order'. Must be 'lat_lon' or 'lon_lat'. Got: {}",
+                geotoh3_argument_order);
     }
 
     std::string getName() const override { return name; }
@@ -94,12 +100,12 @@ class FunctionGeoToH3 : public IFunction
         const ColumnFloat64 * col_lat = nullptr;
         const ColumnFloat64 * col_lon = nullptr;
 
-        if (geotoh3_lon_lat_argument_order)
+        if (geotoh3_argument_order == "lon_lat")
         {
             col_lon = checkAndGetColumn<ColumnFloat64>(non_const_arguments[0].column.get());
             col_lat = checkAndGetColumn<ColumnFloat64>(non_const_arguments[1].column.get());
         }
-        else
+        else // "lat_lon" (setting value is validated in the constructor)
         {
             col_lat = checkAndGetColumn<ColumnFloat64>(non_const_arguments[0].column.get());
             col_lon = checkAndGetColumn<ColumnFloat64>(non_const_arguments[1].column.get());

tests/queries/0_stateless/00937_test_use_header_tsv.sh

@@ -9,7 +9,7 @@ $CLICKHOUSE_CLIENT --query="DROP TABLE IF EXISTS tsv"
 $CLICKHOUSE_CLIENT --query="CREATE TABLE tsv (d Date, u UInt8, str String) ENGINE = TinyLog"
 
 INSERT_QUERY='$CLICKHOUSE_CLIENT --query="INSERT INTO tsv FORMAT TSVWithNames"'
-USE_HEADER='--input_format_with_names_use_header=1'
+USE_HEADER='--input_format_with_names_use_header=1 --output_format_parallel_formatting=1'
 SKIP_UNKNOWN='--input_format_skip_unknown_fields=1'
 
 # Simple check for parsing

tests/queries/0_stateless/03404_geotoh3_input_order.sql

@@ -1,7 +1,7 @@
 -- Tags: no-fasttest
 -- no-fasttest: h3ToGeo needs binary with Uber H3 libary
 
--- Test for setting 'geotoh3_lon_lat_argument_order'
+-- Test for setting 'geotoh3_argument_order'
 
-SELECT geoToH3(37.79506683, 55.71290588, 15) AS h3Index SETTINGS geotoh3_lon_lat_argument_order = true;
-SELECT geoToH3(55.71290588, 37.79506683, 15) AS h3Index SETTINGS geotoh3_lon_lat_argument_order = false;
+SELECT geoToH3(37.79506683, 55.71290588, 15) AS h3Index SETTINGS geotoh3_argument_order = 'lon_lat';
+SELECT geoToH3(55.71290588, 37.79506683, 15) AS h3Index SETTINGS geotoh3_argument_order = 'lat_lon';