Skip to content

Commit 48d4e55

Browse files
prutskovpitrou
authored andcommitted
Fix building (#4)
1 parent 9eed145 commit 48d4e55

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

cpp/src/arrow/csv/converter.cc

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
#include "arrow/type_traits.h"
3333
#include "arrow/util/decimal.h"
3434
#include "arrow/util/parsing.h" // IWYU pragma: keep
35+
#include "arrow/util/timestamp_converter.h"
3536
#include "arrow/util/trie.h"
3637
#include "arrow/util/utf8.h"
3738

@@ -400,7 +401,7 @@ class TimestampConverter : public ConcreteConverter {
400401
builder.UnsafeAppendNull();
401402
return Status::OK();
402403
}
403-
if (!converter(type_, reinterpret_cast<const char*>(data), size, &value)) {
404+
if (!(*converter)(type_, reinterpret_cast<const char*>(data), size, &value)) {
404405
continue;
405406
}
406407
builder.UnsafeAppend(value);

cpp/src/arrow/csv/options.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ struct ARROW_EXPORT ConvertOptions {
101101
/// This option is ignored if `include_columns` is empty.
102102
bool include_missing_columns = false;
103103

104-
std::vector<std::unique_ptr<TimestampConverter>> timestamp_converters;
104+
std::vector<std::shared_ptr<arrow::TimestampConverter>> timestamp_converters;
105105

106106
/// Create conversion options with default values, including conventional
107107
/// values for `null_values`, `true_values` and `false_values`

0 commit comments

Comments
 (0)