Fix tupleConcat of two empty tuples

bigo-sg · Jun 6, 2024 · dd9b15d · dd9b15d
1 parent 6b07873
commit dd9b15d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/src/Functions/tupleConcat.cpp b/src/Functions/tupleConcat.cpp
@@ -61,7 +61,7 @@ class FunctionTupleConcat : public IFunction
         return std::make_shared<DataTypeTuple>(tuple_arg_types);
     }
 
-    ColumnPtr executeImpl(const ColumnsWithTypeAndName & arguments, const DataTypePtr &, size_t) const override
+    ColumnPtr executeImpl(const ColumnsWithTypeAndName & arguments, const DataTypePtr &, size_t input_rows_count) const override
     {
         const size_t num_arguments = arguments.size();
         Columns columns;
@@ -92,6 +92,9 @@ class FunctionTupleConcat : public IFunction
                 columns.push_back(inner_col);
         }
 
+        if (columns.empty())
+            return ColumnTuple::create(input_rows_count);
+
         return ColumnTuple::create(columns);
     }
 };

diff --git a/tests/queries/0_stateless/03167_empty_tuple_concat.reference b/tests/queries/0_stateless/03167_empty_tuple_concat.reference
@@ -0,0 +1 @@
+()
diff --git a/tests/queries/0_stateless/03167_empty_tuple_concat.sql b/tests/queries/0_stateless/03167_empty_tuple_concat.sql
@@ -0,0 +1 @@
+SELECT ()||();