GH-37753: [C++][Gandiva] Add external function registry support #38116
Commits on Oct 24, 2023
-
Add external function registry support to gandiva. A new AddFunction …
…is added to FunctionRegistry so that function metadata can be registered, and external pre-compiled LLVM bitcode/IR buffer can be added so that they can be used for building LLVM module.
-
Add a new class in Gandiva called FunctionRegistar and use it as the …
…dedicated API for registering functions and implementations to gandiva.
-
Refactor the code to return more error when verifying LLVM module in …
…gandiva, and adding some helper functions to reduce code duplication.
-
-
-
-
Use FunctionRegistry as the user facing API for external function reg…
…istry, and make Projector/Filter/LLVMGenerator/Engine to accept a FunctionRegistry for better flexibility.
-
Move function implementation in gandiva test_util.h to test_util.cc t…
…o avoid inlining all functions.
-
Use arrow::Buffer instead of llvm::MemoryBuffer in gandiva FunctionRe…
…gistry so that we don't expose LLVM memory buffer in the header and consumer of FunctionRegistry don't need to be compiled against LLVM headers.
-
Rename TestProjector to non conflict names so that these tests can be…
… added in single add_gandiva_test cmake directive.
-
Use newly created configuration to avoid changing the default configu…
…ration and disable copying and assignment copy for FunctionRegistry since the registration function map contains function signature's address.
-
Change gandiva::default_function_registry() to return shared_ptr inst…
…ead of raw pointer for better ownership.
-
Skip gandiva function registry related tests in c_glib currently and …
…we plan to fix them in a separated PR later.
-
Make sure gandiva filter cache test use a special field name so that …
…the test case's filter cache won't be populated by other test case accidentally.
Commits on Nov 5, 2023
-
Avoid result check when retrieving gandiva default function registry …
…since developers can guarantee they are correct.