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: 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. -- 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