pitrou commented on code in PR #51286:
URL: https://github.com/apache/arrow/pull/51286#discussion_r4410571037
##########
cpp/src/arrow/csv/options.h:
##########
@@ -65,6 +65,8 @@ struct ARROW_EXPORT ParseOptions {
bool pad_short_rows = false;
/// Whether rows with more columns than expected should ignore the extra
columns.
bool ignore_extra_columns = false;
+ /// Multi-character field delimiter. If non-empty, overrides `delimiter`.
+ std::string delimiter_string;
Review Comment:
> In my memory, the reason I didn't deprecate `char delimiter` is that it is
still being used by Python code.
Python shouldn't be a problem as it exposes a Python `str` object regardless
(so we would just relax the 1-character requirement).
However, switching to `std::string` on the C++ side would be a breaking
change. We might want to do it anyway.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
