Skip to content

Commit

Permalink
Use arrow io util to replace std::filesystem::path in gandiva test_ut…
Browse files Browse the repository at this point in the history
…il so that AlmaLinux 8 won't complain.
  • Loading branch information
niyue committed Nov 14, 2023
1 parent 9e5c5b5 commit cf8d98d
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions cpp/src/gandiva/tests/test_util.cc
Original file line number Diff line number Diff line change
Expand Up @@ -17,22 +17,20 @@

#include "gandiva/tests/test_util.h"

#include <filesystem>
#include "arrow/util/io_util.h"

namespace gandiva {
std::shared_ptr<Configuration> TestConfiguration() {
auto builder = ConfigurationBuilder();
return builder.DefaultConfiguration();
return ConfigurationBuilder::DefaultConfiguration();
}

#ifndef GANDIVA_EXTENSION_TEST_DIR
#define GANDIVA_EXTENSION_TEST_DIR "."
#endif

std::string GetTestFunctionLLVMIRPath() {
std::filesystem::path base(GANDIVA_EXTENSION_TEST_DIR);
std::filesystem::path ir_file = base / "multiply_by_two.bc";
return ir_file.string();
const arrow::internal::PlatformFilename base(GANDIVA_EXTENSION_TEST_DIR);
return base.Join("multiply_by_two.bc")->ToString();
}

NativeFunction GetTestExternalFunction() {
Expand Down

0 comments on commit cf8d98d

Please sign in to comment.