Refactor AggregateCompanionAdapter to remove duplicate code

facebookincubator · Jun 3, 2024 · 977ff97 · 977ff97
1 parent 3f39a5b
commit 977ff97
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/velox/exec/AggregateCompanionAdapter.cpp b/velox/exec/AggregateCompanionAdapter.cpp
@@ -471,8 +471,8 @@ bool CompanionFunctionsRegistrar::registerExtractFunctionWithSuffix(
     auto factory = getVectorFunctionFactory(originalName);
     registered |= exec::registerStatefulVectorFunction(
         CompanionSignatures::extractFunctionNameWithSuffix(originalName, type),
-        extractSignatures,
-        factory,
+        std::move(extractSignatures),
+        std::move(factory),
         exec::VectorFunctionMetadataBuilder()
             .defaultNullBehavior(false)
             .build(),
@@ -500,8 +500,8 @@ bool CompanionFunctionsRegistrar::registerExtractFunction(
   auto factory = getVectorFunctionFactory(originalName);
   return exec::registerStatefulVectorFunction(
       CompanionSignatures::extractFunctionName(originalName),
-      extractSignatures,
-      factory,
+      std::move(extractSignatures),
+      std::move(factory),
       exec::VectorFunctionMetadataBuilder().defaultNullBehavior(false).build(),
       overwrite);
 }