From a2b08e064f28b46572d85afe3db53fb387658c04 Mon Sep 17 00:00:00 2001 From: Igor Munkin Date: Wed, 8 May 2024 14:47:44 +0000 Subject: [PATCH] Reorder hints for simple Arrow UDF with optionals To ensure everything works fine as a result of the previous patch, the hints used to create simple Arrow UDFs with optional arguments are reordered in the right way. --- ydb/library/yql/public/udf/arrow/udf_arrow_helpers.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ydb/library/yql/public/udf/arrow/udf_arrow_helpers.h b/ydb/library/yql/public/udf/arrow/udf_arrow_helpers.h index e438c4b3fe89..669e6386b2e0 100644 --- a/ydb/library/yql/public/udf/arrow/udf_arrow_helpers.h +++ b/ydb/library/yql/public/udf/arrow/udf_arrow_helpers.h @@ -623,11 +623,11 @@ class TUnaryOverOptionalImpl : public TBoxedValue { #define BEGIN_SIMPLE_ARROW_UDF_WITH_OPTIONAL_ARGS(udfName, signatureFunc, optArgc) \ BEGIN_ARROW_UDF_IMPL(udfName##_BlocksImpl, signatureFunc, optArgc, false) \ - UDF_IMPL(udfName, builder.SimpleSignature().OptionalArgs(optArgc).SupportsBlocks();, ;, ;, "", "", udfName##_BlocksImpl) + UDF_IMPL(udfName, builder.SimpleSignature().SupportsBlocks().OptionalArgs(optArgc);, ;, ;, "", "", udfName##_BlocksImpl) #define BEGIN_SIMPLE_STRICT_ARROW_UDF_WITH_OPTIONAL_ARGS(udfName, signatureFunc, optArgc) \ BEGIN_ARROW_UDF_IMPL(udfName##_BlocksImpl, signatureFunc, optArgc, true) \ - UDF_IMPL(udfName, builder.SimpleSignature().OptionalArgs(optArgc).SupportsBlocks().IsStrict();, ;, ;, "", "", udfName##_BlocksImpl) + UDF_IMPL(udfName, builder.SimpleSignature().SupportsBlocks().IsStrict().OptionalArgs(optArgc);, ;, ;, "", "", udfName##_BlocksImpl) #define END_ARROW_UDF(udfNameBlocks, exec) \ inline bool udfNameBlocks::DeclareSignature(\