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

Reply via email to