Fix building (apache#4)

prutskov · Apr 13, 2020 · 2f08032 · 2f08032
1 parent bcacc30
commit 2f08032
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/cpp/src/arrow/csv/converter.cc b/cpp/src/arrow/csv/converter.cc
@@ -32,6 +32,7 @@
 #include "arrow/type_traits.h"
 #include "arrow/util/decimal.h"
 #include "arrow/util/parsing.h"  // IWYU pragma: keep
+#include "arrow/util/timestamp_converter.h"
 #include "arrow/util/trie.h"
 #include "arrow/util/utf8.h"
 
@@ -400,7 +401,7 @@ class TimestampConverter : public ConcreteConverter {
             builder.UnsafeAppendNull();
             return Status::OK();
           }
-          if (!converter(type_, reinterpret_cast<const char*>(data), size, &value)) {
+          if (!(*converter)(type_, reinterpret_cast<const char*>(data), size, &value)) {
             continue;
           }
           builder.UnsafeAppend(value);

diff --git a/cpp/src/arrow/csv/options.h b/cpp/src/arrow/csv/options.h
@@ -102,7 +102,7 @@ struct ARROW_EXPORT ConvertOptions {
   /// This option is ignored if `include_columns` is empty.
   bool include_missing_columns = false;
 
-  std::vector<std::unique_ptr<TimestampConverter>> timestamp_converters;
+  std::vector<std::shared_ptr<arrow::TimestampConverter>> timestamp_converters;
 
   /// Create conversion options with default values, including conventional
   /// values for `null_values`, `true_values` and `false_values`