HuaHuaY commented on code in PR #47524:
URL: https://github.com/apache/arrow/pull/47524#discussion_r2343308779
##########
cpp/src/arrow/csv/options.h:
##########
@@ -188,6 +188,12 @@ struct ARROW_EXPORT WriteOptions {
/// Whether to write an initial header line with column names
bool include_header = true;
+ /// \brief Quoting style of header
+ ///
+ /// If `quoting_header` is `QuotingStyle::None`, then only write quotes when
a column
+ /// name contains structural characters. Otherwise, always quote column
names.
+ QuotingStyle quoting_header = QuotingStyle::AllValid;
Review Comment:
To maintain backward compatibility, I have push a new commit to keep the
behavior of header consistent with binary data.
Now, for csv header, `None` will return an error when having structural
charaters. `Needed` and `AllValid` will always write quotes.
There is no such an option which only writes quotes when having structural
charaters (called `Minimal` in the above). We can add the option in the future
if someone needs.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
